Skip to content

tools,test: enable eslint no-unused-vars #4536

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
Trott wants to merge 2 commits into from
Closed

tools,test: enable eslint no-unused-vars #4536

Changes from 1 commit
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
3e65db7
test: remove unused vars
Trott Dec 27, 2015
6235773
tools: implement no-unused-vars for eslint
Trott Dec 23, 2015
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
tools: implement no-unused-vars for eslint
  • Loading branch information
@Trott
Trott committed Jan 5, 2016
commit 623577345b27d4df4314fa7df62716e65f75c90a
2 changes: 2 additions & 0 deletions .eslintrc
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -84,6 +84,8 @@ rules:
# list: https://github.com/eslint/eslint/tree/master/docs/rules#variables
## disallow use of undefined variables (globals)
no-undef: 2
## disallow declaration of variables that are not used in the code
no-unused-vars: [2, {"args": "none"}]

# Custom rules in tools/eslint-rules
require-buffer: 2
Expand Down