Browse Source

tools: enforce two arguments in assert.throws

Enables the requireTwo option of our custom rule.

PR-URL: https://github.com/nodejs/node/pull/12270
Backport-PR-URL: https://github.com/nodejs/node/pull/13785
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
v6.x
Michaël Zasso 8 years ago
committed by Myles Borins
parent
commit
9c67032b9a
No known key found for this signature in database GPG Key ID: 933B01F40B5CA946
  1. 2
      .eslintrc.yaml

2
.eslintrc.yaml

@ -146,7 +146,7 @@ rules:
# Custom rules in tools/eslint-rules # Custom rules in tools/eslint-rules
align-multiline-assignment: 2 align-multiline-assignment: 2
assert-throws-arguments: [2, { requireTwo: false }] assert-throws-arguments: [2, { requireTwo: true }]
no-useless-regex-char-class-escape: [2, { override: ['[', ']'] }] no-useless-regex-char-class-escape: [2, { override: ['[', ']'] }]
# Global scoped method and vars # Global scoped method and vars

Loading…
Cancel
Save