Skip to content

doc: sort repl references in ASCII order - #35230

Merged
Trott merged 1 commit into
nodejs:masterfrom
Trott:repl-sort
Sep 19, 2020
Merged

doc: sort repl references in ASCII order#35230
Trott merged 1 commit into
nodejs:masterfrom
Trott:repl-sort

Conversation

@Trott

Copy link
Copy Markdown
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
Copy Markdown
Contributor

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

@Trott

Copy link
Copy Markdown
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 <daijiro.wachi@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Masashi Hirano <shisama07@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
@Trott

Copy link
Copy Markdown
MemberAuthor

Landed in 4c5bb1a

@Trott
Trott merged commit 4c5bb1a into nodejs:masterSep 19, 2020
@Trott
Trott 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 <daijiro.wachi@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Masashi Hirano <shisama07@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
@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 <daijiro.wachi@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Masashi Hirano <shisama07@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
@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 <daijiro.wachi@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Masashi Hirano <shisama07@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
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