Hi Pedro, On 29/11/2023, 06:40, "Pedro Falcato" <pedro.falc...@gmail.com <mailto:pedro.falc...@gmail.com>> wrote:
On Wed, Nov 29, 2023 at 12:56 AM Ni, Ray <ray...@intel.com <mailto: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. [SAMI] Just saw your email. You have summarised it well. Regards, Sami Mujawar -- Pedro -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#111844): https://edk2.groups.io/g/devel/message/111844 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] -=-=-=-=-=-=-=-=-=-=-=-
