Skip to content

doc: add notable-change to onboarding.md exercise - #21040

Closed
Trott wants to merge 1 commit into
nodejs:masterfrom
Trott:berlin-1
Closed

doc: add notable-change to onboarding.md exercise#21040
Trott wants to merge 1 commit into
nodejs:masterfrom
Trott:berlin-1

Conversation

@Trott

Copy link
Copy Markdown
Member

Pull requests that add new Collaborators should be labeled
notable-change. Make a note of this in onboarding.md.

Checklist

Pull requests that add new Collaborators should be labeled
`notable-change`. Make a note of this in onboarding.md.
@nodejs-github-botnodejs-github-bot added the doc Issues and PRs related to the documentations. label May 30, 2018
@Trott

Trott commented May 30, 2018

Copy link
Copy Markdown
MemberAuthor

@vsemozhetbytvsemozhetbyt added the meta Issues and PRs related to the general management of the project. label May 30, 2018
@TrottTrott added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label May 30, 2018
@nodejsnodejs deleted a comment from refackJun 1, 2018
@Trott

Trott commented Jun 1, 2018

Copy link
Copy Markdown
MemberAuthor

CI failures are not conceivably related but I like my CI green so let's try again: https://ci.nodejs.org/job/node-test-pull-request-lite/783/

Trott added a commit to Trott/io.js that referenced this pull request Jun 2, 2018
Pull requests that add new Collaborators should be labeled
`notable-change`. Make a note of this in onboarding.md.
PR-URL: nodejs#21040
Reviewed-By: Vse Mozhet Byt <vsemozhetbyt@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Refael Ackermann <refack@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
@Trott

Trott commented Jun 2, 2018

Copy link
Copy Markdown
MemberAuthor

Landed in 77d3f14

@TrottTrott closed this Jun 2, 2018
MylesBorins pushed a commit that referenced this pull request Jun 6, 2018
Pull requests that add new Collaborators should be labeled
`notable-change`. Make a note of this in onboarding.md.
PR-URL: #21040
Reviewed-By: Vse Mozhet Byt <vsemozhetbyt@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Refael Ackermann <refack@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
@MylesBorinsMylesBorins mentioned this pull request Jun 6, 2018
@Trott
Trott deleted the berlin-1 branch January 13, 2022 22:49
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author readyPRs that have at least one approval, no pending requests for changes, and a CI started.docIssues and PRs related to the documentations.metaIssues and PRs related to the general management of the project.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants

@Trott@refack@jasnell@lpinca@vsemozhetbyt@trivikr@nodejs-github-bot