On Wed, Mar 23, 2016 at 4:52 PM, Johannes Schindelin <Johannes.Schindelin@xxxxxx> wrote: > Hi Pranit, > > On Tue, 22 Mar 2016, Pranit Bauva wrote: > >> I did run the tests. They produce the same results as they did before. >> To ease review I will next time include these the output of the tests >> in the commented section. >> >> t6002-rev-list-bisect.sh : http://paste.ubuntu.com/15473728/ >> t6030-bisect-porcelain.sh : http://paste.ubuntu.com/15473734/ >> t6041-bisect-submodule.sh : http://paste.ubuntu.com/15473743/ >> >> Is there any other test I would need to run? > > Oh, I just meant to point out that you need to make sure that the entire > test suite passes after your patch series (and ideally, after every patch, > that is at least what I frequently test before sending out patch series). Up to now, I used to only run the tests of the concerned area of the patch. Though from next time, I will take care to run the complete test suite. -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html