Skip to content

Conversation

@BridgeAR
Copy link
Member

@BridgeAR BridgeAR commented Sep 20, 2017

Ref #14167

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
Affected core subsystem(s)

lib

@nodejs-github-bot nodejs-github-bot added lib / src Issues and PRs related to general changes in the lib or src directory. v8.x labels Sep 20, 2017
* internal/errors - assert should already be in place when calling any
  of the message generating functions.
* No lazy load if not necessary.
* Replace function calls with `if`s.

PR-URL: nodejs#14167
Reviewed-By: Refael Ackermann <[email protected]>
Reviewed-By: Timothy Gu <[email protected]>
@BridgeAR BridgeAR force-pushed the backport-14167-to-v8.x branch from d303722 to a3c188e Compare September 21, 2017 01:58
@jasnell
Copy link
Member

jasnell commented Sep 21, 2017

Landed. Thank you!

@jasnell jasnell closed this Sep 21, 2017
@BridgeAR BridgeAR deleted the backport-14167-to-v8.x branch April 1, 2019 23:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lib / src Issues and PRs related to general changes in the lib or src directory.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants