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.
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
Run ESLint on test files, and fix lint errors in test files. #4732
Conversation
Current coverage is 88.81% (diff: 100%)@@ master #4732 diff @@
==========================================
Files 195 195
Lines 13826 13826
Methods 1428 1428
Messages 0 0
Branches 3187 3187
==========================================
Hits 12280 12280
Misses 1546 1546
Partials 0 0
|
@@ -1,5 +1,13 @@ | |||
{ | |||
"extends": "babel", | |||
"globals": { | |||
"test": true, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would these be covered with the mocah env for globals?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ooohh, good call. Pushed the change
'var x = function x(y) {', | ||
' return y;', | ||
'};' | ||
"\"use strict\";", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We can totally use template strings here (different pr)
Awesome this is great! First PR! 🎉 |
* master: (38 commits) chore(package): update browserify to version 13.1.1 (babel#4762) Increase test coverage (babel#4742) Make getBinding ignore labels; add Scope#getLabel, Scope#registerLabel (babel#4758) Add variance node type and generate property variance annotations (babel#4697) Add make command to delete node_modules (babel#4748) fixes [skip ci] Support ObjectExpression in static path evaluation (babel#4746) Fix replacing for-of if inside label (babel#4736) Replace `path-exists` with `fs.existsSync` (babel#4731) Avoid unnecessary +0 in transform-es2015-parameters (babel#4738) [import()] Initial support for dynamic-import (babel#4699) Fix line endings on checkout Automatically generate missing expected.js fixtures (babel#4735) Fix few typos in issue/pr templates (babel#4739) [skip ci] contributing updates [skip ci] increase git depth [skip ci] Change usage of "suite"/"test" in unit-tests to "describe"/"it" (babel#4734) Run ESLint on test files, and fix lint errors in test files (babel#4732) Add .gitattributes forcing LF line endings (babel#4730) Update tests for changed error messages in Babylon (babel#4727) ...
Closes #4714. Updated ESLint globbing patterns in the
makefile
to run overtest
folders now underpackages
, in addition tosrc
. Also ran--fix
to autofix a bunch of errors, and manually fixed the few that aren't autofixable.This change includes adding
some globalsnew env (mocha) globals to be ignored in.eslintrc
. I should note that this will treat those identifiers as globals in all files, not just test files. The alternative to this would be to drop a.eslintrc
in every test directory, or add an ignore comment to the top of each test file, but that seems like it would be a pain longterm.