From: "Neal Gompa" <ngompa13@gmail.com>
To: devel@edk2.groups.io, michael.d.kinney@intel.com
Subject: Re: [edk2-devel] GitHub PR Code Review process now active
Date: Mon, 3 Jun 2024 14:46:30 -0400 [thread overview]
Message-ID: <CAEg-Je_g=Ghm3f4XAjzub-6TvGUNkLQcOCS6Zs5Pyf-R51D9uw@mail.gmail.com> (raw)
In-Reply-To: <CO1PR11MB4929E7232234AB653997F127D2FF2@CO1PR11MB4929.namprd11.prod.outlook.com>
Hmm, I don't see a setting for it anymore, maybe that's not a thing anymore?
I seemingly recall that draft PRs didn't get CI runs, but if that's
not a thing anymore, then that's fine.
That said, draft PRs cannot be reviewed, so we should not be telling
people to make draft PRs.
On Mon, Jun 3, 2024 at 12:26 PM Michael D Kinney via groups.io
<michael.d.kinney=intel.com@groups.io> wrote:
>
> CI jobs are dispatched to both GitHub Actions and Azure Pipelines.
>
> For Draft PRs, I see both GitHub Actions and Azure Pipelines jobs running.
>
> This must imply that edk2 repo allows this. Do you happen to know where
> this is configurable or a link to GitHub docs for configuration?
>
> Mike
>
> > -----Original Message-----
> > From: Neal Gompa <ngompa13@gmail.com>
> > Sent: Monday, June 3, 2024 9:13 AM
> > To: devel@edk2.groups.io; Kinney, Michael D <michael.d.kinney@intel.com>
> > Subject: Re: [edk2-devel] GitHub PR Code Review process now active
> >
> > On Tue, May 28, 2024 at 2:53 PM Michael D Kinney via groups.io
> > <michael.d.kinney=intel.com@groups.io> wrote:
> > >
> > > Hello,
> > >
> > > The GitHub PR code review process is now active. Please
> > > use the new PR based code review process for all new
> > > submissions starting today.
> > >
> > > * The Wiki has been updated with the process changes.
> > >
> > > https://github.com/tianocore/tianocore.github.io/wiki/EDK-II-Development-
> > Process
> > >
> > > Big thanks to Michael Kubacki for writing up all the
> > > changes based on the RFC proposal and community discussions.
> > >
> > > We will learn by using, so if you see anything missing or
> > > incorrect or clarifications needed, please send feedback
> > > here so the Wiki pages can be updated quickly for everyone.
> > >
> > > * The edk2 repo settings have been updated to require
> > > a GitHub PR code review approval before merging and
> > > all conversations must be resolved before merging.
> > >
> > > * A PR has been opened that removes the requirement for
> > > Cc: tags in the commit messages and is the first PR
> > > that will use the new process. This PR needs to be
> > > reviewed and merged to support the revised commit
> > > message format.
> > >
> > > https://github.com/tianocore/edk2/pull/5688
> > >
> > > https://github.com/tianocore/tianocore.github.io/wiki/Commit-Message-
> > Format
> > >
> > > * Please use "Draft" PRs to run CI without any reviews.
> > > Once ready for reviews, convert from "Draft" to
> > > "Ready for Review".
> > >
> >
> > Generally GitHub doesn't allow CI to run on PRs created as draft pull
> > requests. Was this changed for edk2?
> >
> >
> > --
> > 真実はいつも一つ!/ Always, there's only one truth!
>
>
>
>
>
--
真実はいつも一つ!/ Always, there's only one truth!
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#119433): https://edk2.groups.io/g/devel/message/119433
Mute This Topic: https://groups.io/mt/106355103/7686176
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io]
-=-=-=-=-=-=-=-=-=-=-=-
next prev parent reply other threads:[~2024-06-03 18:47 UTC|newest]
Thread overview: 33+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-05-28 18:53 [edk2-devel] GitHub PR Code Review process now active Michael D Kinney
2024-05-29 6:38 ` Gerd Hoffmann
2024-05-29 15:00 ` Michael D Kinney
2024-05-29 16:38 ` Gerd Hoffmann
2024-05-29 18:09 ` Michael D Kinney
2024-05-29 18:18 ` Rebecca Cran
2024-05-29 18:27 ` Michael D Kinney
2024-05-29 19:25 ` Michael Kubacki
2024-05-29 20:06 ` Michael D Kinney
2024-05-30 0:51 ` Michael Kubacki
2024-05-30 17:38 ` Saloni Kasbekar
2024-05-30 18:00 ` Michael D Kinney
2024-05-31 3:50 ` Michael D Kinney
2024-05-31 4:36 ` Michael Kubacki
2024-06-21 5:15 ` Dhaval Sharma
2024-06-21 5:25 ` Michael D Kinney
2024-05-30 8:32 ` Gerd Hoffmann
2024-05-30 14:10 ` Michael D Kinney
2024-05-29 10:40 ` Chang, Abner via groups.io
2024-05-29 14:44 ` Michael D Kinney
2024-05-29 14:48 ` Chang, Abner via groups.io
2024-05-30 0:41 ` Yao, Jiewen
2024-06-03 16:13 ` Neal Gompa
2024-06-03 16:26 ` Michael D Kinney
2024-06-03 18:46 ` Neal Gompa [this message]
2024-06-03 19:38 ` Michael D Kinney
2024-06-05 22:21 ` Michael D Kinney
2024-06-05 22:47 ` Rebecca Cran via groups.io
2024-06-05 23:27 ` Michael D Kinney
2024-06-16 22:08 ` Rebecca Cran
2024-06-06 1:21 ` Guo Dong
2024-06-06 1:37 ` Michael D Kinney
2024-06-04 13:23 ` Gerd Hoffmann
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-list from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to='CAEg-Je_g=Ghm3f4XAjzub-6TvGUNkLQcOCS6Zs5Pyf-R51D9uw@mail.gmail.com' \
--to=devel@edk2.groups.io \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox