Skip to content

errors: keep error codes in alphabetical order - #14242

Closed
starkwang wants to merge 1 commit into
nodejs:masterfrom
starkwang:error-code-order
Closed

errors: keep error codes in alphabetical order#14242
starkwang wants to merge 1 commit into
nodejs:masterfrom
starkwang:error-code-order

Conversation

@starkwang

@starkwangstarkwang commented Jul 15, 2017

Copy link
Copy Markdown
Contributor

The error code ERR_NO_LONGER_SUPPORTED in internal/errors is not in alphabetical order. This PR is to fix it.

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • commit message follows commit guidelines
Affected core subsystem(s)

errors

@nodejs-github-botnodejs-github-bot added the errors Issues and PRs related to JavaScript errors originated in Node.js core. label Jul 15, 2017
Trott
Trott approved these changes Jul 15, 2017

@TrottTrott left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM if CI is green

@vsemozhetbyt

Copy link
Copy Markdown
Contributor

@refackrefack left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

That's the only one. Not too bad, I was sure it was a hot mess.

@refackrefack self-assigned this Jul 16, 2017
@jasnell

Copy link
Copy Markdown
Member

One failure in CI looks unrelated.

@refackrefack removed their assignment Jul 17, 2017
jasnell pushed a commit that referenced this pull request Jul 17, 2017
PR-URL: #14242
Reviewed-By: Vse Mozhet Byt <vsemozhetbyt@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Refael Ackermann <refack@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: James M Snell <jasnell@gmail.com>
@jasnell

Copy link
Copy Markdown
Member

Landed in fa73087

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

errorsIssues and PRs related to JavaScript errors originated in Node.js core.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

11 participants

@starkwang@vsemozhetbyt@jasnell@refack@Trott@lpinca@cjihrig@tniessen@gibfahn@addaleax@nodejs-github-bot