Skip to content

Commit

Permalink
tools, test: require const/let in test
Browse files Browse the repository at this point in the history
PR-URL: #10685
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Roman Reiss <me@silverwind.io>
  • Loading branch information
gibfahn authored and targos committed Jan 28, 2017
1 parent eda18cf commit 06c5604
Showing 1 changed file with 8 additions and 2 deletions.
10 changes: 8 additions & 2 deletions test/.eslintrc.yaml
Original file line number Diff line number Diff line change
@@ -1,7 +1,13 @@
## Test-specific linter rules

rules:
## common module is mandatory in tests
required-modules: [2, common]
# ECMAScript 6
# http://eslint.org/docs/rules/#ecmascript-6
no-var: 2
prefer-const: 2

# Custom rules in tools/eslint-rules
prefer-assert-iferror: 2
prefer-assert-methods: 2
## common module is mandatory in tests
required-modules: [2, common]

0 comments on commit 06c5604

Please sign in to comment.