Skip to content

Conversation

@Trott
Copy link
Member

@TrottTrott commented Sep 3, 2020

Fixes: #35029

Checklist

@nodejs-github-botnodejs-github-bot added doc Issues and PRs related to the documentations. errors Issues and PRs related to JavaScript errors originated in Node.js core. labels Sep 3, 2020
@TrottTrott mentioned this pull request Sep 3, 2020
@Trott
Copy link
MemberAuthor

Trott commented Sep 4, 2020

@nodejs/collaborators This could use reviews.

Copy link
Contributor

@DerekNonGenericDerekNonGeneric left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You're right, it is listed twice.

@lundibundilundibundi added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Sep 5, 2020
@addaleaxaddaleax added the commit-queue Add this label to land a pull request using GitHub Actions. label Sep 5, 2020
@github-actionsgithub-actionsbot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Sep 5, 2020
@github-actions
Copy link
Contributor

Commit Queue failed
- Loading data for nodejs/node/pull/35031 ✔ Done loading data for nodejs/node/pull/35031 ----------------------------------- PR info ------------------------------------ Title doc: remove duplicate error code entry (#35031) Author Rich Trott (@Trott) Branch Trott:dup-err -> nodejs:master Labels author ready, doc, errors Commits 1 - doc: remove duplicate error code entry Committers 1 - Rich Trott PR-URL: https://github.com/nodejs/node/pull/35031 Fixes: https://github.com/nodejs/node/issues/35029 Reviewed-By: Gerhard Stöbich Reviewed-By: Derek Lewis Reviewed-By: Denys Otrishko Reviewed-By: Tobias Nießen ------------------------------ Generated metadata ------------------------------ PR-URL: https://github.com/nodejs/node/pull/35031 Fixes: https://github.com/nodejs/node/issues/35029 Reviewed-By: Gerhard Stöbich Reviewed-By: Derek Lewis Reviewed-By: Denys Otrishko Reviewed-By: Tobias Nießen -------------------------------------------------------------------------------- ✔ Last GitHub Actions successful ℹ Doc-only changes ℹ This PR was created on Thu, 03 Sep 2020 13:59:16 GMT ✔ Approvals: 4 ✔ - Gerhard Stöbich (@Flarna): https://github.com/nodejs/node/pull/35031#pullrequestreview-482946914 ✔ - Derek Lewis (@DerekNonGeneric): https://github.com/nodejs/node/pull/35031#pullrequestreview-482987086 ✔ - Denys Otrishko (@lundibundi): https://github.com/nodejs/node/pull/35031#pullrequestreview-483043807 ✔ - Tobias Nießen (@tniessen) (TSC): https://github.com/nodejs/node/pull/35031#pullrequestreview-483044083 ✖ This PR needs to wait 2 more hours to land -------------------------------------------------------------------------------- ✔ Aborted `git node land` session in /home/runner/work/node/node/.ncu

@github-actionsgithub-actionsbot added the commit-queue-failed An error occurred while landing this pull request using GitHub Actions. label Sep 5, 2020
@addaleaxaddaleax removed the commit-queue-failed An error occurred while landing this pull request using GitHub Actions. label Sep 5, 2020
Fixes: nodejs#35029 PR-URL: nodejs#35031 Reviewed-By: Gerhard Stöbich <[email protected]> Reviewed-By: Derek Lewis <[email protected]> Reviewed-By: Denys Otrishko <[email protected]> Reviewed-By: Tobias Nießen <[email protected]>
@Trott
Copy link
MemberAuthor

Trott commented Sep 5, 2020

Landed in 5e99b62

@TrottTrott merged commit 5e99b62 into nodejs:masterSep 5, 2020
@TrottTrott deleted the dup-err branch September 5, 2020 13:58
richardlau pushed a commit that referenced this pull request Sep 7, 2020
Fixes: #35029 PR-URL: #35031 Reviewed-By: Gerhard Stöbich <[email protected]> Reviewed-By: Derek Lewis <[email protected]> Reviewed-By: Denys Otrishko <[email protected]> Reviewed-By: Tobias Nießen <[email protected]>
@richardlaurichardlau mentioned this pull request Sep 7, 2020
4 tasks
richardlau pushed a commit that referenced this pull request Sep 7, 2020
Fixes: #35029 PR-URL: #35031 Reviewed-By: Gerhard Stöbich <[email protected]> Reviewed-By: Derek Lewis <[email protected]> Reviewed-By: Denys Otrishko <[email protected]> Reviewed-By: Tobias Nießen <[email protected]>
joesepi pushed a commit to joesepi/node that referenced this pull request Jan 8, 2021
Fixes: nodejs#35029 PR-URL: nodejs#35031 Reviewed-By: Gerhard Stöbich <[email protected]> Reviewed-By: Derek Lewis <[email protected]> Reviewed-By: Denys Otrishko <[email protected]> Reviewed-By: Tobias Nießen <[email protected]>
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.errorsIssues and PRs related to JavaScript errors originated in Node.js core.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Duplicate error code

7 participants

@Trott@tniessen@lundibundi@DerekNonGeneric@Flarna@addaleax@nodejs-github-bot