Skip to content

errors: add space between error name and code#12099

Closed
jasnell wants to merge 1 commit into
nodejs:masterfrom
jasnell:errors-tweak
Closed

errors: add space between error name and code#12099
jasnell wants to merge 1 commit into
nodejs:masterfrom
jasnell:errors-tweak

Conversation

@jasnell

@jasnell jasnell commented Mar 28, 2017

Copy link
Copy Markdown
Member

Error[CODE] becomes Error [CODE]

Depends on: #11220

/cc @Trott @evanlucas

(I'm not sure how to classify this on the semver scale since we have not actually used it yet)

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • tests and/or benchmarks are included
  • commit message follows commit guidelines
Affected core subsystem(s)

errors

`Error[CODE]` becomes `Error [CODE]`
@nodejs-github-bot nodejs-github-bot added the errors Issues and PRs related to JavaScript errors originated in Node.js core. label Mar 28, 2017
@jasnell jasnell requested review from Trott and evanlucas March 28, 2017 19:43

@Trott Trott 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.

@gibfahn

gibfahn commented Mar 28, 2017

Copy link
Copy Markdown
Member

(I'm not sure how to classify this on the semver scale since we have not actually used it yet)

seems like a semver-patch then.

@evanlucas evanlucas 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.

I feel like this is better. Thanks!

@addaleax addaleax added this to the 8.0.0 milestone Mar 31, 2017
@addaleax

Copy link
Copy Markdown
Member

jasnell added a commit that referenced this pull request Apr 3, 2017
`Error[CODE]` becomes `Error [CODE]`

PR-URL: #12099
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Evan Lucas <evanlucas@me.com>
@jasnell

jasnell commented Apr 3, 2017

Copy link
Copy Markdown
Member Author

Landed in 7b4a72d

@jasnell jasnell closed this Apr 3, 2017
@jasnell jasnell mentioned this pull request Apr 4, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants