public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
* [edk2-devel] [PATCH edk2-platforms v3 0/1] Move to the GitHub Pull Request workflow
@ 2024-08-01 21:43 Rebecca Cran
  0 siblings, 0 replies; 10+ messages in thread
From: Rebecca Cran @ 2024-08-01 21:43 UTC (permalink / raw)
  To: devel, Leif Lindholm, Michael D Kinney; +Cc: Rebecca Cran

Changes from v2 to v3:

Added note to AssignReviewers.yml to say where it was copied from.

Sorted platforms in CODEOWNERS and REVIEWERS alphabetically.
Sorted entries in CONTRIBUTORS.md by first name.

Updated CONTRIBUTORS.md with some more GitHub usernames.
Updated Readme.md to clarify that people own _areas_ of the repo, not
specifically packages.

Rebecca Cran (1):
  Move to the GitHub Pull Request workflow

 .github/workflows/AssignReviewers.yml |  31 ++
 CODEOWNERS                            | 146 +++++++
 CONTRIBUTORS.md                       |  68 +++
 Maintainers.txt                       | 445 --------------------
 REVIEWERS                             |  92 ++++
 Readme.md                             |  33 +-
 6 files changed, 365 insertions(+), 450 deletions(-)
 create mode 100644 .github/workflows/AssignReviewers.yml
 create mode 100644 CODEOWNERS
 create mode 100644 CONTRIBUTORS.md
 delete mode 100644 Maintainers.txt
 create mode 100644 REVIEWERS

-- 
2.45.2



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#120194): https://edk2.groups.io/g/devel/message/120194
Mute This Topic: https://groups.io/mt/107675097/7686176
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io]
-=-=-=-=-=-=-=-=-=-=-=-



^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [edk2-devel] [PATCH edk2-platforms v3 0/1] Move to the GitHub Pull Request workflow
       [not found] <17E7B8D5FB5780B0.20637@groups.io>
@ 2024-08-06 17:35 ` Rebecca Cran
  2024-08-06 19:14   ` Michael D Kinney
  0 siblings, 1 reply; 10+ messages in thread
From: Rebecca Cran @ 2024-08-06 17:35 UTC (permalink / raw)
  To: devel, Leif Lindholm, Michael D Kinney

Could I get some reviews on this please?


-- 

Rebecca


On 8/1/24 3:43 PM, Rebecca Cran wrote:
> Changes from v2 to v3:
>
> Added note to AssignReviewers.yml to say where it was copied from.
>
> Sorted platforms in CODEOWNERS and REVIEWERS alphabetically.
> Sorted entries in CONTRIBUTORS.md by first name.
>
> Updated CONTRIBUTORS.md with some more GitHub usernames.
> Updated Readme.md to clarify that people own _areas_ of the repo, not
> specifically packages.
>
> Rebecca Cran (1):
>    Move to the GitHub Pull Request workflow
>
>   .github/workflows/AssignReviewers.yml |  31 ++
>   CODEOWNERS                            | 146 +++++++
>   CONTRIBUTORS.md                       |  68 +++
>   Maintainers.txt                       | 445 --------------------
>   REVIEWERS                             |  92 ++++
>   Readme.md                             |  33 +-
>   6 files changed, 365 insertions(+), 450 deletions(-)
>   create mode 100644 .github/workflows/AssignReviewers.yml
>   create mode 100644 CODEOWNERS
>   create mode 100644 CONTRIBUTORS.md
>   delete mode 100644 Maintainers.txt
>   create mode 100644 REVIEWERS
>


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#120256): https://edk2.groups.io/g/devel/message/120256
Mute This Topic: https://groups.io/mt/107675097/7686176
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io]
-=-=-=-=-=-=-=-=-=-=-=-



^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [edk2-devel] [PATCH edk2-platforms v3 0/1] Move to the GitHub Pull Request workflow
  2024-08-06 17:35 ` [edk2-devel] [PATCH edk2-platforms v3 0/1] Move to the GitHub Pull Request workflow Rebecca Cran
@ 2024-08-06 19:14   ` Michael D Kinney
  2024-08-06 20:56     ` Rebecca Cran
  2024-08-19 22:14     ` Rebecca Cran
  0 siblings, 2 replies; 10+ messages in thread
From: Michael D Kinney @ 2024-08-06 19:14 UTC (permalink / raw)
  To: Rebecca Cran, devel@edk2.groups.io, Leif Lindholm,
	Kubacki, Michael
  Cc: Kinney, Michael D

I suspect you will run into the same permissions issues that required
the use of a GitHub App credentials to run the AssignReviewers.yml.

See this change for reference:

	https://github.com/tianocore/edk2/pull/6050


Mike

> -----Original Message-----
> From: Rebecca Cran <rebecca@bsdio.com>
> Sent: Tuesday, August 6, 2024 10:36 AM
> To: devel@edk2.groups.io; Leif Lindholm <quic_llindhol@quicinc.com>; Kinney,
> Michael D <michael.d.kinney@intel.com>
> Subject: Re: [edk2-devel] [PATCH edk2-platforms v3 0/1] Move to the GitHub
> Pull Request workflow
> 
> Could I get some reviews on this please?
> 
> 
> --
> 
> Rebecca
> 
> 
> On 8/1/24 3:43 PM, Rebecca Cran wrote:
> > Changes from v2 to v3:
> >
> > Added note to AssignReviewers.yml to say where it was copied from.
> >
> > Sorted platforms in CODEOWNERS and REVIEWERS alphabetically.
> > Sorted entries in CONTRIBUTORS.md by first name.
> >
> > Updated CONTRIBUTORS.md with some more GitHub usernames.
> > Updated Readme.md to clarify that people own _areas_ of the repo, not
> > specifically packages.
> >
> > Rebecca Cran (1):
> >    Move to the GitHub Pull Request workflow
> >
> >   .github/workflows/AssignReviewers.yml |  31 ++
> >   CODEOWNERS                            | 146 +++++++
> >   CONTRIBUTORS.md                       |  68 +++
> >   Maintainers.txt                       | 445 --------------------
> >   REVIEWERS                             |  92 ++++
> >   Readme.md                             |  33 +-
> >   6 files changed, 365 insertions(+), 450 deletions(-)
> >   create mode 100644 .github/workflows/AssignReviewers.yml
> >   create mode 100644 CODEOWNERS
> >   create mode 100644 CONTRIBUTORS.md
> >   delete mode 100644 Maintainers.txt
> >   create mode 100644 REVIEWERS
> >


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#120257): https://edk2.groups.io/g/devel/message/120257
Mute This Topic: https://groups.io/mt/107675097/7686176
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io]
-=-=-=-=-=-=-=-=-=-=-=-



^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [edk2-devel] [PATCH edk2-platforms v3 0/1] Move to the GitHub Pull Request workflow
  2024-08-06 19:14   ` Michael D Kinney
@ 2024-08-06 20:56     ` Rebecca Cran
  2024-08-06 22:59       ` Michael D Kinney
  2024-08-19 22:14     ` Rebecca Cran
  1 sibling, 1 reply; 10+ messages in thread
From: Rebecca Cran @ 2024-08-06 20:56 UTC (permalink / raw)
  To: Kinney, Michael D, devel@edk2.groups.io, Leif Lindholm,
	Kubacki, Michael

Ah, thanks.

Also, could someone with admin rights please update permissions on 
edk2-platforms so the following maintainers have write access to 
edk2-platforms please?

@zailiangsun

@yqian4

@benjamindoron

@jackpot51

@BobCF

@JohnAZoidberg


-- 

Rebecca


On 8/6/2024 1:14 PM, Kinney, Michael D wrote:
> I suspect you will run into the same permissions issues that required
> the use of a GitHub App credentials to run the AssignReviewers.yml.
>
> See this change for reference:
>
> 	https://github.com/tianocore/edk2/pull/6050
>
>
> Mike
>
>> -----Original Message-----
>> From: Rebecca Cran <rebecca@bsdio.com>
>> Sent: Tuesday, August 6, 2024 10:36 AM
>> To: devel@edk2.groups.io; Leif Lindholm <quic_llindhol@quicinc.com>; Kinney,
>> Michael D <michael.d.kinney@intel.com>
>> Subject: Re: [edk2-devel] [PATCH edk2-platforms v3 0/1] Move to the GitHub
>> Pull Request workflow
>>
>> Could I get some reviews on this please?
>>
>>
>> --
>>
>> Rebecca
>>
>>
>> On 8/1/24 3:43 PM, Rebecca Cran wrote:
>>> Changes from v2 to v3:
>>>
>>> Added note to AssignReviewers.yml to say where it was copied from.
>>>
>>> Sorted platforms in CODEOWNERS and REVIEWERS alphabetically.
>>> Sorted entries in CONTRIBUTORS.md by first name.
>>>
>>> Updated CONTRIBUTORS.md with some more GitHub usernames.
>>> Updated Readme.md to clarify that people own _areas_ of the repo, not
>>> specifically packages.
>>>
>>> Rebecca Cran (1):
>>>     Move to the GitHub Pull Request workflow
>>>
>>>    .github/workflows/AssignReviewers.yml |  31 ++
>>>    CODEOWNERS                            | 146 +++++++
>>>    CONTRIBUTORS.md                       |  68 +++
>>>    Maintainers.txt                       | 445 --------------------
>>>    REVIEWERS                             |  92 ++++
>>>    Readme.md                             |  33 +-
>>>    6 files changed, 365 insertions(+), 450 deletions(-)
>>>    create mode 100644 .github/workflows/AssignReviewers.yml
>>>    create mode 100644 CODEOWNERS
>>>    create mode 100644 CONTRIBUTORS.md
>>>    delete mode 100644 Maintainers.txt
>>>    create mode 100644 REVIEWERS
>>>


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#120258): https://edk2.groups.io/g/devel/message/120258
Mute This Topic: https://groups.io/mt/107675097/7686176
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io]
-=-=-=-=-=-=-=-=-=-=-=-



^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [edk2-devel] [PATCH edk2-platforms v3 0/1] Move to the GitHub Pull Request workflow
  2024-08-06 20:56     ` Rebecca Cran
@ 2024-08-06 22:59       ` Michael D Kinney
  0 siblings, 0 replies; 10+ messages in thread
From: Michael D Kinney @ 2024-08-06 22:59 UTC (permalink / raw)
  To: Rebecca Cran, devel@edk2.groups.io, Leif Lindholm,
	Kubacki, Michael
  Cc: Kinney, Michael D

I have sent invites.  They have to respond to invite.

Mike

> -----Original Message-----
> From: Rebecca Cran <rebecca@bsdio.com>
> Sent: Tuesday, August 6, 2024 1:56 PM
> To: Kinney, Michael D <michael.d.kinney@intel.com>; devel@edk2.groups.io;
> Leif Lindholm <quic_llindhol@quicinc.com>; Kubacki, Michael
> <michael.kubacki@microsoft.com>
> Subject: Re: [edk2-devel] [PATCH edk2-platforms v3 0/1] Move to the GitHub
> Pull Request workflow
> 
> Ah, thanks.
> 
> Also, could someone with admin rights please update permissions on
> edk2-platforms so the following maintainers have write access to
> edk2-platforms please?
> 
> @zailiangsun
> 
> @yqian4
> 
> @benjamindoron
> 
> @jackpot51
> 
> @BobCF
> 
> @JohnAZoidberg
> 
> 
> --
> 
> Rebecca
> 
> 
> On 8/6/2024 1:14 PM, Kinney, Michael D wrote:
> > I suspect you will run into the same permissions issues that required
> > the use of a GitHub App credentials to run the AssignReviewers.yml.
> >
> > See this change for reference:
> >
> > 	https://github.com/tianocore/edk2/pull/6050
> >
> >
> > Mike
> >
> >> -----Original Message-----
> >> From: Rebecca Cran <rebecca@bsdio.com>
> >> Sent: Tuesday, August 6, 2024 10:36 AM
> >> To: devel@edk2.groups.io; Leif Lindholm <quic_llindhol@quicinc.com>;
> Kinney,
> >> Michael D <michael.d.kinney@intel.com>
> >> Subject: Re: [edk2-devel] [PATCH edk2-platforms v3 0/1] Move to the GitHub
> >> Pull Request workflow
> >>
> >> Could I get some reviews on this please?
> >>
> >>
> >> --
> >>
> >> Rebecca
> >>
> >>
> >> On 8/1/24 3:43 PM, Rebecca Cran wrote:
> >>> Changes from v2 to v3:
> >>>
> >>> Added note to AssignReviewers.yml to say where it was copied from.
> >>>
> >>> Sorted platforms in CODEOWNERS and REVIEWERS alphabetically.
> >>> Sorted entries in CONTRIBUTORS.md by first name.
> >>>
> >>> Updated CONTRIBUTORS.md with some more GitHub usernames.
> >>> Updated Readme.md to clarify that people own _areas_ of the repo, not
> >>> specifically packages.
> >>>
> >>> Rebecca Cran (1):
> >>>     Move to the GitHub Pull Request workflow
> >>>
> >>>    .github/workflows/AssignReviewers.yml |  31 ++
> >>>    CODEOWNERS                            | 146 +++++++
> >>>    CONTRIBUTORS.md                       |  68 +++
> >>>    Maintainers.txt                       | 445 --------------------
> >>>    REVIEWERS                             |  92 ++++
> >>>    Readme.md                             |  33 +-
> >>>    6 files changed, 365 insertions(+), 450 deletions(-)
> >>>    create mode 100644 .github/workflows/AssignReviewers.yml
> >>>    create mode 100644 CODEOWNERS
> >>>    create mode 100644 CONTRIBUTORS.md
> >>>    delete mode 100644 Maintainers.txt
> >>>    create mode 100644 REVIEWERS
> >>>


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#120259): https://edk2.groups.io/g/devel/message/120259
Mute This Topic: https://groups.io/mt/107675097/7686176
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io]
-=-=-=-=-=-=-=-=-=-=-=-



^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [edk2-devel] [PATCH edk2-platforms v3 0/1] Move to the GitHub Pull Request workflow
  2024-08-06 19:14   ` Michael D Kinney
  2024-08-06 20:56     ` Rebecca Cran
@ 2024-08-19 22:14     ` Rebecca Cran
  2024-08-19 23:01       ` Michael D Kinney
  1 sibling, 1 reply; 10+ messages in thread
From: Rebecca Cran @ 2024-08-19 22:14 UTC (permalink / raw)
  To: Kinney, Michael D, devel@edk2.groups.io, Leif Lindholm,
	Kubacki, Michael

Could you explain what you entered into the New GitHub App page when 
setting up tianocore to use 
https://github.com/marketplace/actions/create-github-app-token please?


-- 

Rebecca


On 8/6/24 1:14 PM, Kinney, Michael D wrote:
> I suspect you will run into the same permissions issues that required
> the use of a GitHub App credentials to run the AssignReviewers.yml.
>
> See this change for reference:
>
> 	https://github.com/tianocore/edk2/pull/6050
>
>
> Mike
>
>> -----Original Message-----
>> From: Rebecca Cran <rebecca@bsdio.com>
>> Sent: Tuesday, August 6, 2024 10:36 AM
>> To: devel@edk2.groups.io; Leif Lindholm <quic_llindhol@quicinc.com>; Kinney,
>> Michael D <michael.d.kinney@intel.com>
>> Subject: Re: [edk2-devel] [PATCH edk2-platforms v3 0/1] Move to the GitHub
>> Pull Request workflow
>>
>> Could I get some reviews on this please?
>>
>>
>> --
>>
>> Rebecca
>>
>>
>> On 8/1/24 3:43 PM, Rebecca Cran wrote:
>>> Changes from v2 to v3:
>>>
>>> Added note to AssignReviewers.yml to say where it was copied from.
>>>
>>> Sorted platforms in CODEOWNERS and REVIEWERS alphabetically.
>>> Sorted entries in CONTRIBUTORS.md by first name.
>>>
>>> Updated CONTRIBUTORS.md with some more GitHub usernames.
>>> Updated Readme.md to clarify that people own _areas_ of the repo, not
>>> specifically packages.
>>>
>>> Rebecca Cran (1):
>>>     Move to the GitHub Pull Request workflow
>>>
>>>    .github/workflows/AssignReviewers.yml |  31 ++
>>>    CODEOWNERS                            | 146 +++++++
>>>    CONTRIBUTORS.md                       |  68 +++
>>>    Maintainers.txt                       | 445 --------------------
>>>    REVIEWERS                             |  92 ++++
>>>    Readme.md                             |  33 +-
>>>    6 files changed, 365 insertions(+), 450 deletions(-)
>>>    create mode 100644 .github/workflows/AssignReviewers.yml
>>>    create mode 100644 CODEOWNERS
>>>    create mode 100644 CONTRIBUTORS.md
>>>    delete mode 100644 Maintainers.txt
>>>    create mode 100644 REVIEWERS
>>>


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#120367): https://edk2.groups.io/g/devel/message/120367
Mute This Topic: https://groups.io/mt/107675097/7686176
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io]
-=-=-=-=-=-=-=-=-=-=-=-



^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [edk2-devel] [PATCH edk2-platforms v3 0/1] Move to the GitHub Pull Request workflow
  2024-08-19 22:14     ` Rebecca Cran
@ 2024-08-19 23:01       ` Michael D Kinney
  2024-08-19 23:30         ` Rebecca Cran
  0 siblings, 1 reply; 10+ messages in thread
From: Michael D Kinney @ 2024-08-19 23:01 UTC (permalink / raw)
  To: Rebecca Cran, devel@edk2.groups.io, Leif Lindholm,
	Kubacki, Michael
  Cc: Kinney, Michael D

You do not need a new app.

We can use the app that is already available.

The main update that is needed is adding secrets to edk2-platforms repo

Mike

> -----Original Message-----
> From: Rebecca Cran <rebecca@bsdio.com>
> Sent: Monday, August 19, 2024 3:15 PM
> To: Kinney, Michael D <michael.d.kinney@intel.com>; devel@edk2.groups.io;
> Leif Lindholm <quic_llindhol@quicinc.com>; Kubacki, Michael
> <michael.kubacki@microsoft.com>
> Subject: Re: [edk2-devel] [PATCH edk2-platforms v3 0/1] Move to the GitHub
> Pull Request workflow
> 
> Could you explain what you entered into the New GitHub App page when
> setting up tianocore to use
> https://github.com/marketplace/actions/create-github-app-token please?
> 
> 
> --
> 
> Rebecca
> 
> 
> On 8/6/24 1:14 PM, Kinney, Michael D wrote:
> > I suspect you will run into the same permissions issues that required
> > the use of a GitHub App credentials to run the AssignReviewers.yml.
> >
> > See this change for reference:
> >
> > 	https://github.com/tianocore/edk2/pull/6050
> >
> >
> > Mike
> >
> >> -----Original Message-----
> >> From: Rebecca Cran <rebecca@bsdio.com>
> >> Sent: Tuesday, August 6, 2024 10:36 AM
> >> To: devel@edk2.groups.io; Leif Lindholm <quic_llindhol@quicinc.com>;
> Kinney,
> >> Michael D <michael.d.kinney@intel.com>
> >> Subject: Re: [edk2-devel] [PATCH edk2-platforms v3 0/1] Move to the
> GitHub
> >> Pull Request workflow
> >>
> >> Could I get some reviews on this please?
> >>
> >>
> >> --
> >>
> >> Rebecca
> >>
> >>
> >> On 8/1/24 3:43 PM, Rebecca Cran wrote:
> >>> Changes from v2 to v3:
> >>>
> >>> Added note to AssignReviewers.yml to say where it was copied from.
> >>>
> >>> Sorted platforms in CODEOWNERS and REVIEWERS alphabetically.
> >>> Sorted entries in CONTRIBUTORS.md by first name.
> >>>
> >>> Updated CONTRIBUTORS.md with some more GitHub usernames.
> >>> Updated Readme.md to clarify that people own _areas_ of the repo, not
> >>> specifically packages.
> >>>
> >>> Rebecca Cran (1):
> >>>     Move to the GitHub Pull Request workflow
> >>>
> >>>    .github/workflows/AssignReviewers.yml |  31 ++
> >>>    CODEOWNERS                            | 146 +++++++
> >>>    CONTRIBUTORS.md                       |  68 +++
> >>>    Maintainers.txt                       | 445 --------------------
> >>>    REVIEWERS                             |  92 ++++
> >>>    Readme.md                             |  33 +-
> >>>    6 files changed, 365 insertions(+), 450 deletions(-)
> >>>    create mode 100644 .github/workflows/AssignReviewers.yml
> >>>    create mode 100644 CODEOWNERS
> >>>    create mode 100644 CONTRIBUTORS.md
> >>>    delete mode 100644 Maintainers.txt
> >>>    create mode 100644 REVIEWERS
> >>>


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#120369): https://edk2.groups.io/g/devel/message/120369
Mute This Topic: https://groups.io/mt/107675097/7686176
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io]
-=-=-=-=-=-=-=-=-=-=-=-



^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [edk2-devel] [PATCH edk2-platforms v3 0/1] Move to the GitHub Pull Request workflow
  2024-08-19 23:01       ` Michael D Kinney
@ 2024-08-19 23:30         ` Rebecca Cran
  2024-08-20  1:58           ` Michael D Kinney
  0 siblings, 1 reply; 10+ messages in thread
From: Rebecca Cran @ 2024-08-19 23:30 UTC (permalink / raw)
  To: Kinney, Michael D, devel@edk2.groups.io, Leif Lindholm,
	Kubacki, Michael

Sorry, I should have explained: I'd like to set it up on 
https://github.com/bcran/edk2-platforms to test it before having the 
changes merged into https://github.com/tianocore/edk2-platforms.


-- 

Rebecca


On 8/19/2024 5:01 PM, Kinney, Michael D wrote:
> You do not need a new app.
>
> We can use the app that is already available.
>
> The main update that is needed is adding secrets to edk2-platforms repo
>
> Mike
>
>> -----Original Message-----
>> From: Rebecca Cran <rebecca@bsdio.com>
>> Sent: Monday, August 19, 2024 3:15 PM
>> To: Kinney, Michael D <michael.d.kinney@intel.com>; devel@edk2.groups.io;
>> Leif Lindholm <quic_llindhol@quicinc.com>; Kubacki, Michael
>> <michael.kubacki@microsoft.com>
>> Subject: Re: [edk2-devel] [PATCH edk2-platforms v3 0/1] Move to the GitHub
>> Pull Request workflow
>>
>> Could you explain what you entered into the New GitHub App page when
>> setting up tianocore to use
>> https://github.com/marketplace/actions/create-github-app-token please?
>>
>>
>> --
>>
>> Rebecca
>>
>>
>> On 8/6/24 1:14 PM, Kinney, Michael D wrote:
>>> I suspect you will run into the same permissions issues that required
>>> the use of a GitHub App credentials to run the AssignReviewers.yml.
>>>
>>> See this change for reference:
>>>
>>> 	https://github.com/tianocore/edk2/pull/6050
>>>
>>>
>>> Mike
>>>
>>>> -----Original Message-----
>>>> From: Rebecca Cran <rebecca@bsdio.com>
>>>> Sent: Tuesday, August 6, 2024 10:36 AM
>>>> To: devel@edk2.groups.io; Leif Lindholm <quic_llindhol@quicinc.com>;
>> Kinney,
>>>> Michael D <michael.d.kinney@intel.com>
>>>> Subject: Re: [edk2-devel] [PATCH edk2-platforms v3 0/1] Move to the
>> GitHub
>>>> Pull Request workflow
>>>>
>>>> Could I get some reviews on this please?
>>>>
>>>>
>>>> --
>>>>
>>>> Rebecca
>>>>
>>>>
>>>> On 8/1/24 3:43 PM, Rebecca Cran wrote:
>>>>> Changes from v2 to v3:
>>>>>
>>>>> Added note to AssignReviewers.yml to say where it was copied from.
>>>>>
>>>>> Sorted platforms in CODEOWNERS and REVIEWERS alphabetically.
>>>>> Sorted entries in CONTRIBUTORS.md by first name.
>>>>>
>>>>> Updated CONTRIBUTORS.md with some more GitHub usernames.
>>>>> Updated Readme.md to clarify that people own _areas_ of the repo, not
>>>>> specifically packages.
>>>>>
>>>>> Rebecca Cran (1):
>>>>>      Move to the GitHub Pull Request workflow
>>>>>
>>>>>     .github/workflows/AssignReviewers.yml |  31 ++
>>>>>     CODEOWNERS                            | 146 +++++++
>>>>>     CONTRIBUTORS.md                       |  68 +++
>>>>>     Maintainers.txt                       | 445 --------------------
>>>>>     REVIEWERS                             |  92 ++++
>>>>>     Readme.md                             |  33 +-
>>>>>     6 files changed, 365 insertions(+), 450 deletions(-)
>>>>>     create mode 100644 .github/workflows/AssignReviewers.yml
>>>>>     create mode 100644 CODEOWNERS
>>>>>     create mode 100644 CONTRIBUTORS.md
>>>>>     delete mode 100644 Maintainers.txt
>>>>>     create mode 100644 REVIEWERS
>>>>>


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#120371): https://edk2.groups.io/g/devel/message/120371
Mute This Topic: https://groups.io/mt/107675097/7686176
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io]
-=-=-=-=-=-=-=-=-=-=-=-



^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [edk2-devel] [PATCH edk2-platforms v3 0/1] Move to the GitHub Pull Request workflow
  2024-08-19 23:30         ` Rebecca Cran
@ 2024-08-20  1:58           ` Michael D Kinney
  2024-08-20  2:04             ` Michael Kubacki
  0 siblings, 1 reply; 10+ messages in thread
From: Michael D Kinney @ 2024-08-20  1:58 UTC (permalink / raw)
  To: Rebecca Cran, devel@edk2.groups.io, Leif Lindholm,
	Kubacki, Michael
  Cc: Kinney, Michael D

Here is the guide I used that includes the GitHub App approach

* https://michaelheap.com/ultimate-guide-github-actions-authentication/

Mike

> -----Original Message-----
> From: Rebecca Cran <rebecca@bsdio.com>
> Sent: Monday, August 19, 2024 4:31 PM
> To: Kinney, Michael D <michael.d.kinney@intel.com>; devel@edk2.groups.io;
> Leif Lindholm <quic_llindhol@quicinc.com>; Kubacki, Michael
> <michael.kubacki@microsoft.com>
> Subject: Re: [edk2-devel] [PATCH edk2-platforms v3 0/1] Move to the GitHub
> Pull Request workflow
> 
> Sorry, I should have explained: I'd like to set it up on
> https://github.com/bcran/edk2-platforms to test it before having the
> changes merged into https://github.com/tianocore/edk2-platforms.
> 
> 
> --
> 
> Rebecca
> 
> 
> On 8/19/2024 5:01 PM, Kinney, Michael D wrote:
> > You do not need a new app.
> >
> > We can use the app that is already available.
> >
> > The main update that is needed is adding secrets to edk2-platforms repo
> >
> > Mike
> >
> >> -----Original Message-----
> >> From: Rebecca Cran <rebecca@bsdio.com>
> >> Sent: Monday, August 19, 2024 3:15 PM
> >> To: Kinney, Michael D <michael.d.kinney@intel.com>;
> devel@edk2.groups.io;
> >> Leif Lindholm <quic_llindhol@quicinc.com>; Kubacki, Michael
> >> <michael.kubacki@microsoft.com>
> >> Subject: Re: [edk2-devel] [PATCH edk2-platforms v3 0/1] Move to the
> GitHub
> >> Pull Request workflow
> >>
> >> Could you explain what you entered into the New GitHub App page when
> >> setting up tianocore to use
> >> https://github.com/marketplace/actions/create-github-app-token please?
> >>
> >>
> >> --
> >>
> >> Rebecca
> >>
> >>
> >> On 8/6/24 1:14 PM, Kinney, Michael D wrote:
> >>> I suspect you will run into the same permissions issues that required
> >>> the use of a GitHub App credentials to run the AssignReviewers.yml.
> >>>
> >>> See this change for reference:
> >>>
> >>> 	https://github.com/tianocore/edk2/pull/6050
> >>>
> >>>
> >>> Mike
> >>>
> >>>> -----Original Message-----
> >>>> From: Rebecca Cran <rebecca@bsdio.com>
> >>>> Sent: Tuesday, August 6, 2024 10:36 AM
> >>>> To: devel@edk2.groups.io; Leif Lindholm <quic_llindhol@quicinc.com>;
> >> Kinney,
> >>>> Michael D <michael.d.kinney@intel.com>
> >>>> Subject: Re: [edk2-devel] [PATCH edk2-platforms v3 0/1] Move to the
> >> GitHub
> >>>> Pull Request workflow
> >>>>
> >>>> Could I get some reviews on this please?
> >>>>
> >>>>
> >>>> --
> >>>>
> >>>> Rebecca
> >>>>
> >>>>
> >>>> On 8/1/24 3:43 PM, Rebecca Cran wrote:
> >>>>> Changes from v2 to v3:
> >>>>>
> >>>>> Added note to AssignReviewers.yml to say where it was copied from.
> >>>>>
> >>>>> Sorted platforms in CODEOWNERS and REVIEWERS alphabetically.
> >>>>> Sorted entries in CONTRIBUTORS.md by first name.
> >>>>>
> >>>>> Updated CONTRIBUTORS.md with some more GitHub usernames.
> >>>>> Updated Readme.md to clarify that people own _areas_ of the repo, not
> >>>>> specifically packages.
> >>>>>
> >>>>> Rebecca Cran (1):
> >>>>>      Move to the GitHub Pull Request workflow
> >>>>>
> >>>>>     .github/workflows/AssignReviewers.yml |  31 ++
> >>>>>     CODEOWNERS                            | 146 +++++++
> >>>>>     CONTRIBUTORS.md                       |  68 +++
> >>>>>     Maintainers.txt                       | 445 --------------------
> >>>>>     REVIEWERS                             |  92 ++++
> >>>>>     Readme.md                             |  33 +-
> >>>>>     6 files changed, 365 insertions(+), 450 deletions(-)
> >>>>>     create mode 100644 .github/workflows/AssignReviewers.yml
> >>>>>     create mode 100644 CODEOWNERS
> >>>>>     create mode 100644 CONTRIBUTORS.md
> >>>>>     delete mode 100644 Maintainers.txt
> >>>>>     create mode 100644 REVIEWERS
> >>>>>


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#120372): https://edk2.groups.io/g/devel/message/120372
Mute This Topic: https://groups.io/mt/107675097/7686176
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io]
-=-=-=-=-=-=-=-=-=-=-=-



^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [edk2-devel] [PATCH edk2-platforms v3 0/1] Move to the GitHub Pull Request workflow
  2024-08-20  1:58           ` Michael D Kinney
@ 2024-08-20  2:04             ` Michael Kubacki
  0 siblings, 0 replies; 10+ messages in thread
From: Michael Kubacki @ 2024-08-20  2:04 UTC (permalink / raw)
  To: devel, michael.d.kinney, Rebecca Cran, Leif Lindholm,
	Kubacki, Michael

For simplicity of testing on the fork, I'd just create a PAT in your 
repo with the necessary permissions and use that as the secret there. 
Then, substitute it with the app secret in the edk2-platforms PR.

You can test almost everything that way except the app token generation 
itself. But, that's mostly copy/paste. Here's an example of the change 
to switch from a normal secret token, whether the default token as shown 
there, or a PAT secret, to an app token:

https://github.com/tianocore/edk2/commit/1b37b3659b5098f764dee5b893e4eb174949f40a

On 8/19/2024 9:58 PM, Michael D Kinney wrote:
> Here is the guide I used that includes the GitHub App approach
> 
> * https://michaelheap.com/ultimate-guide-github-actions-authentication/
> 
> Mike
> 
>> -----Original Message-----
>> From: Rebecca Cran <rebecca@bsdio.com>
>> Sent: Monday, August 19, 2024 4:31 PM
>> To: Kinney, Michael D <michael.d.kinney@intel.com>; devel@edk2.groups.io;
>> Leif Lindholm <quic_llindhol@quicinc.com>; Kubacki, Michael
>> <michael.kubacki@microsoft.com>
>> Subject: Re: [edk2-devel] [PATCH edk2-platforms v3 0/1] Move to the GitHub
>> Pull Request workflow
>>
>> Sorry, I should have explained: I'd like to set it up on
>> https://github.com/bcran/edk2-platforms to test it before having the
>> changes merged into https://github.com/tianocore/edk2-platforms.
>>
>>
>> --
>>
>> Rebecca
>>
>>
>> On 8/19/2024 5:01 PM, Kinney, Michael D wrote:
>>> You do not need a new app.
>>>
>>> We can use the app that is already available.
>>>
>>> The main update that is needed is adding secrets to edk2-platforms repo
>>>
>>> Mike
>>>
>>>> -----Original Message-----
>>>> From: Rebecca Cran <rebecca@bsdio.com>
>>>> Sent: Monday, August 19, 2024 3:15 PM
>>>> To: Kinney, Michael D <michael.d.kinney@intel.com>;
>> devel@edk2.groups.io;
>>>> Leif Lindholm <quic_llindhol@quicinc.com>; Kubacki, Michael
>>>> <michael.kubacki@microsoft.com>
>>>> Subject: Re: [edk2-devel] [PATCH edk2-platforms v3 0/1] Move to the
>> GitHub
>>>> Pull Request workflow
>>>>
>>>> Could you explain what you entered into the New GitHub App page when
>>>> setting up tianocore to use
>>>> https://github.com/marketplace/actions/create-github-app-token please?
>>>>
>>>>
>>>> --
>>>>
>>>> Rebecca
>>>>
>>>>
>>>> On 8/6/24 1:14 PM, Kinney, Michael D wrote:
>>>>> I suspect you will run into the same permissions issues that required
>>>>> the use of a GitHub App credentials to run the AssignReviewers.yml.
>>>>>
>>>>> See this change for reference:
>>>>>
>>>>> 	https://github.com/tianocore/edk2/pull/6050
>>>>>
>>>>>
>>>>> Mike
>>>>>
>>>>>> -----Original Message-----
>>>>>> From: Rebecca Cran <rebecca@bsdio.com>
>>>>>> Sent: Tuesday, August 6, 2024 10:36 AM
>>>>>> To: devel@edk2.groups.io; Leif Lindholm <quic_llindhol@quicinc.com>;
>>>> Kinney,
>>>>>> Michael D <michael.d.kinney@intel.com>
>>>>>> Subject: Re: [edk2-devel] [PATCH edk2-platforms v3 0/1] Move to the
>>>> GitHub
>>>>>> Pull Request workflow
>>>>>>
>>>>>> Could I get some reviews on this please?
>>>>>>
>>>>>>
>>>>>> --
>>>>>>
>>>>>> Rebecca
>>>>>>
>>>>>>
>>>>>> On 8/1/24 3:43 PM, Rebecca Cran wrote:
>>>>>>> Changes from v2 to v3:
>>>>>>>
>>>>>>> Added note to AssignReviewers.yml to say where it was copied from.
>>>>>>>
>>>>>>> Sorted platforms in CODEOWNERS and REVIEWERS alphabetically.
>>>>>>> Sorted entries in CONTRIBUTORS.md by first name.
>>>>>>>
>>>>>>> Updated CONTRIBUTORS.md with some more GitHub usernames.
>>>>>>> Updated Readme.md to clarify that people own _areas_ of the repo, not
>>>>>>> specifically packages.
>>>>>>>
>>>>>>> Rebecca Cran (1):
>>>>>>>       Move to the GitHub Pull Request workflow
>>>>>>>
>>>>>>>      .github/workflows/AssignReviewers.yml |  31 ++
>>>>>>>      CODEOWNERS                            | 146 +++++++
>>>>>>>      CONTRIBUTORS.md                       |  68 +++
>>>>>>>      Maintainers.txt                       | 445 --------------------
>>>>>>>      REVIEWERS                             |  92 ++++
>>>>>>>      Readme.md                             |  33 +-
>>>>>>>      6 files changed, 365 insertions(+), 450 deletions(-)
>>>>>>>      create mode 100644 .github/workflows/AssignReviewers.yml
>>>>>>>      create mode 100644 CODEOWNERS
>>>>>>>      create mode 100644 CONTRIBUTORS.md
>>>>>>>      delete mode 100644 Maintainers.txt
>>>>>>>      create mode 100644 REVIEWERS
>>>>>>>
> 
> 
> 
> 
> 



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#120373): https://edk2.groups.io/g/devel/message/120373
Mute This Topic: https://groups.io/mt/107675097/7686176
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io]
-=-=-=-=-=-=-=-=-=-=-=-



^ permalink raw reply	[flat|nested] 10+ messages in thread

end of thread, other threads:[~2024-08-20  2:04 UTC | newest]

Thread overview: 10+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
     [not found] <17E7B8D5FB5780B0.20637@groups.io>
2024-08-06 17:35 ` [edk2-devel] [PATCH edk2-platforms v3 0/1] Move to the GitHub Pull Request workflow Rebecca Cran
2024-08-06 19:14   ` Michael D Kinney
2024-08-06 20:56     ` Rebecca Cran
2024-08-06 22:59       ` Michael D Kinney
2024-08-19 22:14     ` Rebecca Cran
2024-08-19 23:01       ` Michael D Kinney
2024-08-19 23:30         ` Rebecca Cran
2024-08-20  1:58           ` Michael D Kinney
2024-08-20  2:04             ` Michael Kubacki
2024-08-01 21:43 Rebecca Cran

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox