FYI: 'make check' is running in pipelines for merge requests since this morning 🙂
One thing I forgot in that lengthy email two weeks ago was how MRs get to Patch::review. Traditionally James posted "Passes make check" and updated the labels accordingly. As 'make check' is now happening automatically, I think it makes sense to distribute this job: Basically once the pipeline is green and 'make check' produced *something* to look at, every member of the group passing by may update the labels. My gut feeling is that this should be somebody else than the MR author, but I'm not sold on that. Furthermore, it might be nice to make a screenshot of the visual difference and post that and the link for discussion, which makes it much easier to spot (tree.gittxt and the spurious errors about "Parsed object should be dead" can be skipped). WDYT? For the record, moving to Patch::countdown and ::push as well as composing the countdown messages would remain on James as the Patch Meister. Jonas
signature.asc
Description: This is a digitally signed message part
