On Wed, Nov 29, 2023 at 12:56 AM Ni, Ray <ray...@intel.com> wrote: > > It's good. But I am curious why --ignore-change-id is needed?
I didn't ask, but presumably, if you have an internal gerrit instance that runs CI before pushing, PatchCheck.py may be part of the CI workflow; in those cases, we don't want it to error out. So the CI would be adapted to do PatchCheck.py --ignore-change-id, and all is well. -- Pedro -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#111836): https://edk2.groups.io/g/devel/message/111836 Mute This Topic: https://groups.io/mt/102748141/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
