Skip to content

update link in onboarding doc - #40539

Merged
Trott merged 1 commit into
masterfrom
Trott-patch-1
Oct 22, 2021
Merged

update link in onboarding doc#40539
Trott merged 1 commit into
masterfrom
Trott-patch-1

Conversation

@Trott

Copy link
Copy Markdown
Member

No description provided.

@TrottTrott added the fast-track PRs that do not need to wait for 48 hours to land. label Oct 21, 2021
@github-actions

Copy link
Copy Markdown
Contributor

Fast-track has been requested by @Trott. Please 👍 to approve.

@nodejs-github-botnodejs-github-bot added the doc Issues and PRs related to the documentations. label Oct 21, 2021
@mscdex

Copy link
Copy Markdown
Contributor

s/onbaording/onboarding/ in commit message

@TrottTrott changed the title update link in onbaording docupdate link in onboarding docOct 21, 2021
@richardlaurichardlau added the commit-queue Add this label to land a pull request using GitHub Actions. label Oct 21, 2021
@github-actionsgithub-actionsBot added commit-queue-failed An error occurred while landing this pull request using GitHub Actions. and removed commit-queue Add this label to land a pull request using GitHub Actions. labels Oct 21, 2021
@github-actions

Copy link
Copy Markdown
Contributor
Commit Queue failed
- Loading data for nodejs/node/pull/40539
✔ Done loading data for nodejs/node/pull/40539
----------------------------------- PR info ------------------------------------
Title update link in onboarding doc (#40539)
Author Rich Trott (@Trott)
Branch Trott:Trott-patch-1 -> nodejs:master
Labels doc, fast-track
Commits 1
- doc: update link in onboarding doc
Committers 1
- Rich Trott PR-URL: https://github.com/nodejs/node/pull/40539
Reviewed-By: Benjamin Gruenbaum Reviewed-By: Richard Lau ------------------------------ Generated metadata ------------------------------
PR-URL: https://github.com/nodejs/node/pull/40539
Reviewed-By: Benjamin Gruenbaum Reviewed-By: Richard Lau --------------------------------------------------------------------------------
ℹ This PR was created on Thu, 21 Oct 2021 02:01:29 GMT
✔ Approvals: 2
✔ - Benjamin Gruenbaum (@benjamingr): https://github.com/nodejs/node/pull/40539#pullrequestreview-785423419
✔ - Richard Lau (@richardlau) (TSC): https://github.com/nodejs/node/pull/40539#pullrequestreview-785495130
ℹ This PR is being fast-tracked
✖ GitHub CI is still running
ℹ Green GitHub Actions CI is sufficient
--------------------------------------------------------------------------------
✔ Aborted `git node land` session in /home/runner/work/node/node/.ncu
https://github.com/nodejs/node/actions/runs/1367452046

@MesteeryMesteery added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Oct 21, 2021
@TrottTrott added commit-queue Add this label to land a pull request using GitHub Actions. and removed commit-queue-failed An error occurred while landing this pull request using GitHub Actions. labels Oct 21, 2021
@github-actionsgithub-actionsBot added commit-queue-failed An error occurred while landing this pull request using GitHub Actions. and removed commit-queue Add this label to land a pull request using GitHub Actions. labels Oct 21, 2021
@github-actions

Copy link
Copy Markdown
Contributor
Commit Queue failed
- Loading data for nodejs/node/pull/40539
✔ Done loading data for nodejs/node/pull/40539
----------------------------------- PR info ------------------------------------
Title update link in onboarding doc (#40539)
Author Rich Trott (@Trott)
Branch Trott:Trott-patch-1 -> nodejs:master
Labels doc, fast-track, author ready
Commits 1
- doc: update link in onboarding doc
Committers 1
- Rich Trott PR-URL: https://github.com/nodejs/node/pull/40539
Reviewed-By: Benjamin Gruenbaum Reviewed-By: Richard Lau Reviewed-By: Colin Ihrig Reviewed-By: Tobias Nießen Reviewed-By: Luigi Pinca ------------------------------ Generated metadata ------------------------------
PR-URL: https://github.com/nodejs/node/pull/40539
Reviewed-By: Benjamin Gruenbaum Reviewed-By: Richard Lau Reviewed-By: Colin Ihrig Reviewed-By: Tobias Nießen Reviewed-By: Luigi Pinca --------------------------------------------------------------------------------
ℹ This PR was created on Thu, 21 Oct 2021 02:01:29 GMT
✔ Approvals: 5
✔ - Benjamin Gruenbaum (@benjamingr): https://github.com/nodejs/node/pull/40539#pullrequestreview-785423419
✔ - Richard Lau (@richardlau) (TSC): https://github.com/nodejs/node/pull/40539#pullrequestreview-785495130
✔ - Colin Ihrig (@cjihrig) (TSC): https://github.com/nodejs/node/pull/40539#pullrequestreview-785826483
✔ - Tobias Nießen (@tniessen) (TSC): https://github.com/nodejs/node/pull/40539#pullrequestreview-785858072
✔ - Luigi Pinca (@lpinca): https://github.com/nodejs/node/pull/40539#pullrequestreview-786091650
ℹ This PR is being fast-tracked
✖ GitHub CI is still running
ℹ Green GitHub Actions CI is sufficient
--------------------------------------------------------------------------------
✔ Aborted `git node land` session in /home/runner/work/node/node/.ncu
https://github.com/nodejs/node/actions/runs/1369553323

PR-URL: #40539
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
@Trott
Trottforce-pushed the Trott-patch-1 branch 2 times, most recently from 7853180 to 0d59d0dCompareOctober 22, 2021 05:11
@Trott

Copy link
Copy Markdown
MemberAuthor

Landed in 0d59d0d

@Trott
Trott merged commit 0d59d0d into masterOct 22, 2021
@Trott
Trott deleted the Trott-patch-1 branch October 22, 2021 05:12
targos pushed a commit that referenced this pull request Oct 23, 2021
PR-URL: #40539
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
@targostargos mentioned this pull request Nov 8, 2021
BethGriggs pushed a commit that referenced this pull request Nov 23, 2021
PR-URL: #40539
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
@BethGriggsBethGriggs mentioned this pull request Nov 26, 2021
1 task
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.commit-queue-failedAn error occurred while landing this pull request using GitHub Actions.docIssues and PRs related to the documentations.fast-trackPRs that do not need to wait for 48 hours to land.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

10 participants

@Trott@mscdex@benjamingr@lpinca@cjihrig@tniessen@richardlau@J-Human@nodejs-github-bot@Mesteery