Skip to content

Commit

Permalink
tools: enforce two arguments in assert.throws
Browse files Browse the repository at this point in the history
Enables the requireTwo option of our custom rule.

PR-URL: #12270
Backport-PR-URL: #13785
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
  • Loading branch information
targos authored and gibfahn committed Jun 20, 2017
1 parent 63c3e6b commit 77b4095
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion .eslintrc.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -146,7 +146,7 @@ rules:

# Custom rules in tools/eslint-rules
align-multiline-assignment: 2
assert-throws-arguments: [2, { requireTwo: false }]
assert-throws-arguments: [2, { requireTwo: true }]
no-useless-regex-char-class-escape: [2, { override: ['[', ']'] }]

# Global scoped method and vars
Expand Down

0 comments on commit 77b4095

Please sign in to comment.