Skip to content

doc: add missing pull request number - #27336

Closed
BridgeAR wants to merge 1 commit into
nodejs:masterfrom
BridgeAR:fix-changes-entry
Closed

doc: add missing pull request number#27336
BridgeAR wants to merge 1 commit into
nodejs:masterfrom
BridgeAR:fix-changes-entry

Conversation

@BridgeAR

Copy link
Copy Markdown
Member

This adds a missing pull request number to the repl.md.

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • tests and/or benchmarks are included
  • documentation is changed or added
  • commit message follows commit guidelines

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-botnodejs-github-bot added doc Issues and PRs related to the documentations. repl Issues and PRs related to the REPL subsystem. labels Apr 21, 2019
@vsemozhetbyt

Copy link
Copy Markdown
Contributor

Node.js Collaborators, please, add 👍 here if you approve fast-tracking.

@vsemozhetbytvsemozhetbyt added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Apr 21, 2019
@vsemozhetbyt

vsemozhetbyt commented Apr 21, 2019

Copy link
Copy Markdown
Contributor

Is failing test due to a race condition with these PRs: #25947 and #27151 ?

@mscdex

Copy link
Copy Markdown
Contributor

This PR seems to add a lot more than just a PR number?

@BridgeAR

Copy link
Copy Markdown
MemberAuthor

@mscdex I had to force push to master to fix a test and I did not yet update this PR (see the note about the force push). It's only about the second commit.

@BridgeAR

Copy link
Copy Markdown
MemberAuthor

@ZYSzys

Copy link
Copy Markdown
Member

@BridgeAR Seems that there are some conflicts need to be resolved.

This adds a missing pull request number to the `repl.md`.
@BridgeAR

Copy link
Copy Markdown
MemberAuthor

BridgeAR added a commit to BridgeAR/node that referenced this pull request Apr 24, 2019
This fixes a pull request number in `repl.md`.
PR-URL: nodejs#27336
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Vse Mozhet Byt <vsemozhetbyt@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Yongsheng Zhang <zyszys98@gmail.com>
@BridgeAR

Copy link
Copy Markdown
MemberAuthor

Landed in 237d9f9 🎉

(I updated the description while landing since the PR number was actually set to a falsy value in the meanwhile).

targos pushed a commit that referenced this pull request Apr 27, 2019
This fixes a pull request number in `repl.md`.
PR-URL: #27336
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Vse Mozhet Byt <vsemozhetbyt@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Yongsheng Zhang <zyszys98@gmail.com>
@targostargos mentioned this pull request Apr 27, 2019
@BridgeAR
BridgeAR deleted the fix-changes-entry branch January 20, 2020 12:02
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.replIssues and PRs related to the REPL subsystem.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

10 participants

@BridgeAR@nodejs-github-bot@vsemozhetbyt@mscdex@ZYSzys@addaleax@benjamingr@lpinca@tniessen@mihalskiy