Skip to content

Conversation

@Trott
Copy link
Member

Checklist

@nodejs-github-botnodejs-github-bot added doc Issues and PRs related to the documentations. repl Issues and PRs related to the REPL subsystem. labels Sep 16, 2020
@aduh95
Copy link
Contributor

If that's something we want to enforce, should we create a test for this?

@Trott
Copy link
MemberAuthor

If that's something we want to enforce, should we create a test for this?

A test or a lint rule would be great IMO. Every time I open one of these PRs, I think about a lint rule, but realistically I don't think I'll ever actually do it. So if you feel motivated, by all means, please do!

PR-URL: nodejs#35230 Reviewed-By: Daijiro Wachi <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Masashi Hirano <[email protected]> Reviewed-By: Trivikram Kamat <[email protected]> Reviewed-By: Anna Henningsen <[email protected]>
@Trott
Copy link
MemberAuthor

Landed in 4c5bb1a

@TrottTrott merged commit 4c5bb1a into nodejs:masterSep 19, 2020
@TrottTrott deleted the repl-sort branch September 19, 2020 13:09
ruyadorno pushed a commit that referenced this pull request Sep 21, 2020
PR-URL: #35230 Reviewed-By: Daijiro Wachi <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Masashi Hirano <[email protected]> Reviewed-By: Trivikram Kamat <[email protected]> Reviewed-By: Anna Henningsen <[email protected]>
@ruyadornoruyadorno mentioned this pull request Sep 21, 2020
4 tasks
addaleax pushed a commit that referenced this pull request Sep 22, 2020
PR-URL: #35230 Reviewed-By: Daijiro Wachi <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Masashi Hirano <[email protected]> Reviewed-By: Trivikram Kamat <[email protected]> Reviewed-By: Anna Henningsen <[email protected]>
@codebyterecodebytere mentioned this pull request Sep 28, 2020
joesepi pushed a commit to joesepi/node that referenced this pull request Jan 8, 2021
PR-URL: nodejs#35230 Reviewed-By: Daijiro Wachi <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Masashi Hirano <[email protected]> Reviewed-By: Trivikram Kamat <[email protected]> Reviewed-By: Anna Henningsen <[email protected]>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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.

9 participants

@Trott@aduh95@addaleax@lpinca@watilde@shisama@trivikr@MrHeer@nodejs-github-bot