Re: [edk2-devel] github impact:breaking-change

2024-07-09 Thread Michael Kubacki
The tweaks look good to me. Thanks, Michael On 7/9/2024 11:03 AM, Leif Lindholm wrote: Hi Michael, On Tue, Jul 09, 2024 at 10:44:46 -0400, Michael Kubacki wrote: Hi Leif, Thanks for raising this. The label was intended to be set from the PR template which has an explanation: https://github.

Re: [edk2-devel] github impact:breaking-change

2024-07-09 Thread Leif Lindholm
Hi Michael, On Tue, Jul 09, 2024 at 10:44:46 -0400, Michael Kubacki wrote: > Hi Leif, > > Thanks for raising this. The label was intended to be set from the PR > template which has an explanation: > > https://github.com/tianocore/edk2/blob/master/.github/pull_request_template.md > > If the brea

Re: [edk2-devel] github impact:breaking-change

2024-07-09 Thread Michael Kubacki
Hi Leif, Thanks for raising this. The label was intended to be set from the PR template which has an explanation: https://github.com/tianocore/edk2/blob/master/.github/pull_request_template.md If the breaking change box is checked, it will be added automatically. I added a description in Git

[edk2-devel] github impact:breaking-change

2024-07-08 Thread Leif Lindholm
Hi, I'm seeing something in several PRs in flight (and merged) that have "impact:breaking-change" set, where the purpose of the PR is to fix said breakage, not introduce API compatibilities. Am I correct in my understanding that this is not the intended use, and if so how do we address the m