Hi Laszlo,

===================================================
Note for all reviewers:
---------------------------------------------------
Pull requests against edk2-staging/edk2-ci are not
being processed right now.  We are working on some
configuration changes after noticing that all the
checks were not being shown on the Web UI.  We will
let you know when it is back up.
===================================================

Yes.  You need to create a fork of the tianocore/edk2-staging repo.

You can do this with the WebUI or the hub command line utility.

* https://github.com/github/hub
* https://github.com/github/hub/releases
* https://hub.github.com/hub.1.html

The hub command also supports creating a pull request.
I have used it extensively to write some unit tests
for edk2-ci this week.

Once you create a branch with changes to submit in your
own fork of tianocore/edk2-staging the WebUI will show
that this pull request is possible and guide you through
it.

The use of edk2-staging/edk2-ci is only for the review
and unit testing.  Once the review is approved, it will
be enabled on edk2/master and you will be able to use
your own fork of edk2 to make branches and submit pull
requests.

Mike

> -----Original Message-----
> From: Laszlo Ersek <ler...@redhat.com>
> Sent: Thursday, October 31, 2019 2:55 AM
> To: devel@edk2.groups.io; Kinney, Michael D
> <michael.d.kin...@intel.com>
> Cc: Sean Brogan <sean.bro...@microsoft.com>; Bret
> Barkelew <bret.barke...@microsoft.com>; Gao, Liming
> <liming....@intel.com>; Feng, Bob C
> <bob.c.f...@intel.com>; Andrew Fish <af...@apple.com>;
> Leif Lindholm <leif.lindh...@linaro.org>; Wang, Jian J
> <jian.j.w...@intel.com>; Lu, XiaoyuX
> <xiaoyux...@intel.com>; Ni, Ray <ray...@intel.com>; Wu,
> Hao A <hao.a...@intel.com>; Wu, Jiaxin
> <jiaxin...@intel.com>; Fu, Siyuan <siyuan...@intel.com>;
> Yao, Jiewen <jiewen....@intel.com>; Zhang, Chao B
> <chao.b.zh...@intel.com>; Gao, Zhichao
> <zhichao....@intel.com>; Dong, Eric
> <eric.d...@intel.com>
> Subject: Re: [edk2-devel] [Patch v3 00/22] Enable Phase
> 1 of EDK II CI
> 
> On 10/29/19 20:54, Michael D Kinney wrote:
> 
> > Active branch for testing/evaluation:
> > * https://github.com/tianocore/edk2-staging/tree/edk2-
> ci
> > * To test, fork edk2-staging repo, create a branch
> with a change, and submit
> >   a pull request targeting edk2-staging/edk2-ci.
> NOTE: the default branch for
> >   the edk2-staging is 'about'.  You must select the
> 'edk2-ci' branch when
> >   a pull request is opened.  Set the 'push' label to
> require commit if all
> >   checks pass.
> 
> The edk2-staging repository has been added as a "git
> remote" to my local
> edk2 clone for a long while now. Using the local
> identifier "staging".
> (This makes perfect sense as edk2-staging is itself a
> fork of edk2, with branches that are supposed to be
> rebased to edk2/master periodically.)
> 
> Furthermore, the identifier by which I refer to the
> remote at <https://github.com/lersek/edk2.git> is
> "lersek".
> 
> I've now run the following commands:
> 
> $ git fetch staging
> $ git checkout -b ci-test-1 staging/edk2-ci
> 
> [modify "SampleFile.txt"]
> 
> $ git add -p
> $ git commit
> $ git push lersek ci-test-1
> 
> Questions:
> 
> (a) How can I submit a pull request for the staging
> repo's edk2-ci branch using the command line (and set
> the "push" label)?
> 
> (b) How can I submit a PR for the staging repo's edk2-ci
> branch (regardless of command line vs. WebUI usage)
> against my <https://github.com/lersek/edk2.git>
> repository?
> 
> When I go to the WebUI, the PR view does not offer
> "tianocore/edk2-staging" as "base repository", it only
> offers "tianocore/edk2". I thought I'd be able to pick
> any destination repository at all.
> 
> By the instruction "fork edk2-staging repo", did you
> mean we should fork edk2-staging *on github*? (Using the
> WebUI?)
> 
> Thanks!
> Laszlo


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.

View/Reply Online (#49792): https://edk2.groups.io/g/devel/message/49792
Mute This Topic: https://groups.io/mt/39614149/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

Reply via email to