On Tue, Feb 9, 2016 at 10:42 AM, Junio C Hamano <gitster@xxxxxxxxx> wrote: > Lars Schneider <larsxschneider@xxxxxxxxx> writes: > >> Jeff Merkey made me aware of http://kernelnewbies.org/FirstKernelPatch [2] >> where I found checkpatch.pl [3]. Would it make sense to check all commits >> that are not in next/master/maint with this script on Travis-CI? > > That does not help very much. These changes are already shown to > people and dirtied their eyes, and most likely I've already have > wasted time tweaking the glitches out locally. The damage has > already been done. > > It would make a lot of sense if the checkpatch is called inside > Roberto Tyley's "pull-request-to-patch-submission" thing, though. Last time I reported a bug/feature request, Roberto was glad to be cc'd as he doesn't read the mailing list in full. So I cc'd him here. -- 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