On Thu, 15 Sept 2022 at 22:52, Kinney, Michael D <michael.d.kin...@intel.com> wrote: > > Ard, > > Why would you want to do that? The whole point of CI is to establish a > minimum quality level for all code in the project. > > They can be disabled with updates to the YAML file. Checks can be disabled > completely and may of the checks support exception lists. >
If the only way to prevent this from happening is to turn it off again in the YAML file, I'd prefer not to turn it on to begin with. I agree that code quality is important, but IMO the checks we have at the moment are way too strict, and 90% of the time I spend on reviewing and merging patches is on crustify and patchcheck errors. This is simply not worth my time. -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#93873): https://edk2.groups.io/g/devel/message/93873 Mute This Topic: https://groups.io/mt/93517518/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
