On Tue, Dec 22, 2020 at 02:56:19PM -0500, Don Zickus wrote: > On Tue, Dec 22, 2020 at 01:17:43PM -0500, Prarit Bhargava wrote: > > It seems that cki-bot doesn't like it when I remove the "No CCs" label from > > an > > MR. You can see this odd behaviour in !741: > > > > https://gitlab.com/cki-project/kernel-ark/-/merge_requests/741 > > > > I ran a script to identify all the mislabeled "No CCs" MRs, and tried to > > remove > > it by hand from !741. The cki-bot immediately steps in and relabels it. > > > > Is there some step I'm missing? !741 clearly has CCs. The only thing I can > > think of is that cki-bot's webhook is broken. > > OK, I think I know what the problem is (/me loves the --merge-request hook > for webhooks). The public webhooks walks all the commits in the MR. If 1 > of them does not have CCs, then it slaps a label on it. > > In this, Patrick's added patch is missing the CC. Of course he doesn't care > because it gets squashed later. But that is what is triggering it. > > We could change the code to say if at least 1 _has_ a CC, ignore it and move > on. Or perhaps we should only look for CCs in the merge log instead of the > commit log? > > The label is specific to changes in 'redhat/configs/[fedora|ark|common]', so > perhaps just the merge message is good enough???
Err. Nope. That was the how the original public webhook worked until someone started posting commits with empty merge request descriptions. *cough*Prarit*cough* So the other option is to flip it from 'if 1 does _not_ have Ccs' to 'if _none_ have CCs', then set the NO_CCS_LABEL. Thoughts? Cheers, Don _______________________________________________ kernel mailing list -- [email protected] To unsubscribe send an email to [email protected] Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedoraproject.org/archives/list/[email protected]
