Skip to content

doc: clarify the review and landing process #10202

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
joyeecheung wants to merge 10 commits into from
Closed

doc: clarify the review and landing process #10202

Changes from 1 commit
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
cc22d35
doc: clarify the review and landing process
joyeecheung Dec 9, 2016
e8153b7
doc: clarify the landing process
joyeecheung Dec 11, 2016
0bb76ba
doc: explain the waiting time of semver-major PRs
joyeecheung Dec 13, 2016
32ab27a
doc: put the notes and details in the back
joyeecheung Dec 14, 2016
9bda4a4
doc: improve the clarification
joyeecheung Dec 15, 2016
dbd83ef
doc: more fixes about the process
joyeecheung Dec 15, 2016
96e0189
doc: amends the contributing guide
joyeecheung Dec 16, 2016
246f52a
doc: testing docs, PR history preservation
joyeecheung Dec 16, 2016
e9c53f3
doc: fix typo
joyeecheung Dec 20, 2016
41e4e93
doc: trim trailling spaces
joyeecheung Dec 22, 2016
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
doc: clarify the landing process 
- Mention the "landed in <sha>" comment when a PR gets landed
  • Loading branch information
@joyeecheung
joyeecheung committed Dec 11, 2016
commit e8153b72a665ae54f6ddeef25cc22f7ea90f7e06
8 changes: 5 additions & 3 deletions CONTRIBUTING.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -289,9 +289,11 @@ Collaborators and has passed the CI, as long as there is consensus
But, if there are non-trivial changes in this Pull Request, it still
needs to wait for at least another 48 hours (72 hours on a weekend).

GitHub often shows the Pull Request as `Closed` at this point,
but don't worry. If you look at the branch you raised your Pull Request
against (probably `master`), you should see a commit with
When the Pull Request is landed, the person who land this will post
a comment to the Pull Request page mentioning the commit(s) in which
it is landed. GitHub often shows the Pull Request as `Closed` at this
point, but don't worry. If you look at the branch you raised your
Pull Request against (probably `master`), you should see a commit with
your name on it. Congratulations and thanks for your contribution!

<a id="developers-certificate-of-origin"></a>
Expand Down