> On Jun 20, 2018, at 8:15 AM, Aaron Conole <[email protected]> wrote: > > 0-day Robot <[email protected]> writes: > >> Bleep bloop. Greetings Justin Pettit, I am a robot and I have tried out >> your patch >> with message ID <[email protected]> >> Thanks for your contribution. >> >> I encountered some error that I wasn't expecting. See the details below. >> >> >> checkpatch: >> == Checking "0000.patch" == >> ERROR: Too many signoffs; are you missing Co-authored-by lines? >> Lines checked: 118, Warnings: 0, Errors: 1 > > I'll fix the sign off check in checkpatch. It shouldn't flag on these > kinds of patches. > > Sorry for the noise.
No problem. Thanks for setting it up. I agree that it shouldn't flag on this issue, however, the bigger concern I have is that the warning seems to be separated from the file names, which is a bit confusing. And, I imagine it would be very confusing if someone had multiple patches flagged, since it wouldn't be clear which error list applied to which patch. --Justin _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
