Skip to content
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.

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

doc: remove duplicate error code entry #35031

Merged
merged 1 commit into from
Sep 5, 2020
Merged

doc: remove duplicate error code entry #35031

merged 1 commit into from
Sep 5, 2020

Conversation

Trott
Copy link
Member

@Trott Trott commented Sep 3, 2020

Fixes: #35029

Checklist

@nodejs-github-bot nodejs-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
@Trott Trott mentioned this pull request Sep 3, 2020
@Trott
Copy link
Member Author

Trott commented Sep 4, 2020

@nodejs/collaborators This could use reviews.

Copy link
Contributor

@DerekNonGeneric DerekNonGeneric 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.

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

github-actions bot commented Sep 5, 2020

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-actions github-actions bot added the commit-queue-failed An error occurred while landing this pull request using GitHub Actions. label Sep 5, 2020
@addaleax addaleax 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
Member Author

Trott commented Sep 5, 2020

Landed in 5e99b62

@Trott Trott merged commit 5e99b62 into nodejs:master Sep 5, 2020
@Trott Trott 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]>
@richardlau richardlau 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 free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
author ready PRs that have at least one approval, no pending requests for changes, and a CI started. doc Issues and PRs related to the documentations. errors Issues 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