Skip to content

doc: remove duplicated sentences in large-pull-requests.md#63650

Merged
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
joyeecheung:large-pr-fix
May 30, 2026
Merged

doc: remove duplicated sentences in large-pull-requests.md#63650
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
joyeecheung:large-pr-fix

Conversation

@joyeecheung
Copy link
Copy Markdown
Member

@joyeecheung joyeecheung commented May 29, 2026

I think it probably came from bugs in the GitHub suggestion UI..(funny enough, the duplicated sentences are also citing issues in the GitHub UI 😅)

Signed-off-by: Joyee Cheung <joyeec9h3@gmail.com>
@nodejs-github-bot
Copy link
Copy Markdown
Collaborator

Review requested:

  • @nodejs/tsc

@nodejs-github-bot nodejs-github-bot added the doc Issues and PRs related to the documentations. label May 29, 2026
@MikeMcC399

This comment was marked as off-topic.

@joyeecheung joyeecheung added the fast-track PRs that do not need to wait for 72 hours to land. label May 29, 2026
@github-actions
Copy link
Copy Markdown
Contributor

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

Copy link
Copy Markdown
Member

@mcollina mcollina left a comment

Choose a reason for hiding this comment

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

lgtm

@mcollina mcollina added the commit-queue Add this label to land a pull request using GitHub Actions. label May 30, 2026
@nodejs-github-bot nodejs-github-bot 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 May 30, 2026
@nodejs-github-bot
Copy link
Copy Markdown
Collaborator

Commit Queue failed
- Loading data for nodejs/node/pull/63650
✔  Done loading data for nodejs/node/pull/63650
----------------------------------- PR info ------------------------------------
Title      doc: remove duplicated sentences in large-pull-requests.md (#63650)
Author     Joyee Cheung <joyeec9h3@gmail.com> (@joyeecheung)
Branch     joyeecheung:large-pr-fix -> nodejs:main
Labels     doc, fast-track
Commits    1
 - doc: remove duplicated sentences in large-pull-requests.md
Committers 1
 - Joyee Cheung <joyeec9h3@gmail.com>
PR-URL: https://github.com/nodejs/node/pull/63650
Reviewed-By: Filip Skokan <panva.ip@gmail.com>
Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
------------------------------ Generated metadata ------------------------------
PR-URL: https://github.com/nodejs/node/pull/63650
Reviewed-By: Filip Skokan <panva.ip@gmail.com>
Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
--------------------------------------------------------------------------------
   ℹ  This PR was created on Fri, 29 May 2026 18:39:30 GMT
   ✔  Approvals: 5
   ✔  - Filip Skokan (@panva) (TSC): https://github.com/nodejs/node/pull/63650#pullrequestreview-4392009190
   ✔  - Rafael Gonzaga (@RafaelGSS) (TSC): https://github.com/nodejs/node/pull/63650#pullrequestreview-4392303617
   ✔  - Matteo Collina (@mcollina) (TSC): https://github.com/nodejs/node/pull/63650#pullrequestreview-4392832022
   ✔  - Tobias Nießen (@tniessen) (TSC): https://github.com/nodejs/node/pull/63650#pullrequestreview-4393021742
   ✔  - Trivikram Kamat (@trivikr): https://github.com/nodejs/node/pull/63650#pullrequestreview-4394197948
   ℹ  This PR is being fast-tracked
   ✘  This PR needs to wait 35 more hours to land (or 0 hours if there is 1 more approval (👍) of the fast-track request from collaborators).
   ✔  Last GitHub CI successful
   ℹ  Green GitHub CI is sufficient
--------------------------------------------------------------------------------
   ✔  Aborted `git node land` session in /home/runner/work/node/node/.ncu
https://github.com/nodejs/node/actions/runs/26678843884

@aduh95 aduh95 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 May 30, 2026
@nodejs-github-bot nodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label May 30, 2026
@nodejs-github-bot nodejs-github-bot merged commit 80e5174 into nodejs:main May 30, 2026
36 checks passed
@nodejs-github-bot
Copy link
Copy Markdown
Collaborator

Landed in 80e5174

aduh95 pushed a commit that referenced this pull request May 30, 2026
Signed-off-by: Joyee Cheung <joyeec9h3@gmail.com>
PR-URL: #63650
Reviewed-By: Filip Skokan <panva.ip@gmail.com>
Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

doc Issues and PRs related to the documentations. fast-track PRs that do not need to wait for 72 hours to land.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

10 participants