diff options
author | Ruben Bridgewater <ruben@bridgewater.de> | 2019-10-03 02:24:19 +0200 |
---|---|---|
committer | Rich Trott <rtrott@gmail.com> | 2019-10-05 20:19:17 -0700 |
commit | 0521a98fd6dfbd31994709f0f417c470f98d2b71 (patch) | |
tree | 4614a56a95c7be9aa64030206e6e163c7405de4a /lib | |
parent | 870eea303ad4f7d821b558d0f6fefbe738c33afc (diff) | |
download | android-node-v8-0521a98fd6dfbd31994709f0f417c470f98d2b71.tar.gz android-node-v8-0521a98fd6dfbd31994709f0f417c470f98d2b71.tar.bz2 android-node-v8-0521a98fd6dfbd31994709f0f417c470f98d2b71.zip |
tools: prohibit Error.prepareStackTrace() usage
This eslint rule makes sure that `prepareStackTrace()` is not used in
Node.js core.
PR-URL: https://github.com/nodejs/node/pull/29827
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Gus Caplan <me@gus.host>
Diffstat (limited to 'lib')
-rw-r--r-- | lib/.eslintrc.yaml | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/lib/.eslintrc.yaml b/lib/.eslintrc.yaml index b7e8e1f4fb..f66e15e6d9 100644 --- a/lib/.eslintrc.yaml +++ b/lib/.eslintrc.yaml @@ -33,6 +33,8 @@ rules: message: "Use an error exported by the internal/errors module." - selector: "CallExpression[callee.object.name='Error'][callee.property.name='captureStackTrace']" message: "Please use `require('internal/errors').hideStackFrames()` instead." + - selector: "AssignmentExpression:matches([left.name='prepareStackTrace'], [left.property.name='prepareStackTrace'])" + message: "Use 'overrideStackTrace' from 'lib/internal/errors.js' instead of 'Error.prepareStackTrace'." # Custom rules in tools/eslint-rules node-core/lowercase-name-for-primitive: error node-core/non-ascii-character: error |