From: Torsten Bögershausen Date: Sun, 09 Oct 2016 05:42:18 GMT Subject: Re: [PATCH v10 14/14] contrib/long-running-filter: add long running filter example Message-ID: <3bdfebab-aae9-3263-218e-c8ae394084fc@web.de> In-Reply-To: <20161008112530.15506-15-larsxschneider@gmail.com> On 08.10.16 13:25, larsxschneider@gmail.com wrote: > From: Lars Schneider > > Add a simple pass-thru filter as example implementation for the Git > filter protocol version 2. See Documentation/gitattributes.txt, section > "Filter Protocol" for more info. > Nothing wrong with code in contrib. I may have missed parts of the discussion, was there a good reason to drop the test case completely? >When adding a new feature, make sure that you have new tests to show >the feature triggers the new behavior when it should, and to show the >feature does not trigger when it shouldn't. After any code change, make >sure that the entire test suite passes. Or is there a plan to add them later ?