mirror of
https://github.com/google/blockly.git
synced 2026-01-10 02:17:09 +01:00
chore(deps): Remove dependency on babel-eslint; simplify eslintrc for mocha tests (#5794)
* chore(deps): Remove deprecated babel-lint package When dependabot submitted PR #5791, it became apparent that eslint version >= 8 is incompatible with babel-eslint v10.1.0 despite the latter not having a version restriction against it. This is probably because v10.1.0 is the last version of the package before it was deprecated and renamed @babel/eslint-parser. A little more investigation determined that there is no longer any need to have eslint use a non-default parser for tests/mocha/**/*.js, so the most straight forward solution is just to remove the dependency entirely. * refactor(tests): Simplify tests/mocha/.eslintrc.json Remove all rules that seem to be redundant with our top-level .eslintrc.json. This finishes reverting most of the changes made in PR #4062.
This commit is contained in:
committed by
GitHub
parent
6ed2b6a31f
commit
ec6bf32ade
@@ -1,5 +1,4 @@
|
||||
{
|
||||
"parser": "babel-eslint",
|
||||
"env": {
|
||||
"browser": true,
|
||||
"mocha": true
|
||||
@@ -11,30 +10,8 @@
|
||||
},
|
||||
"rules": {
|
||||
"no-unused-vars": ["off"],
|
||||
"es5/no-arrow-functions": ["off"],
|
||||
"es5/no-binary-and-octal-literals": ["off"],
|
||||
"es5/no-block-scoping": ["off"],
|
||||
"es5/no-classes": ["off"],
|
||||
"es5/no-computed-properties": ["off"],
|
||||
"es5/no-default-parameters": ["off"],
|
||||
"es5/no-destructuring": ["off"],
|
||||
"es5/no-es6-methods": ["off"],
|
||||
"es5/no-es6-static-methods": ["off"],
|
||||
"es5/no-for-of": ["off"],
|
||||
"es5/no-generators": ["off"],
|
||||
"es5/no-modules": ["off"],
|
||||
"es5/no-object-super": ["off"],
|
||||
"es5/no-rest-parameters": ["off"],
|
||||
"es5/no-shorthand-properties": ["off"],
|
||||
"es5/no-spread": ["off"],
|
||||
"es5/no-template-literals": ["off"],
|
||||
"es5/no-typeof-symbol": ["off"],
|
||||
"es5/no-unicode-code-point-escape": ["off"],
|
||||
"es5/no-unicode-regex": ["off"],
|
||||
// Allow uncommented helper functions in tests.
|
||||
"require-jsdoc": ["off"],
|
||||
// In mocha tests in suites, `this` is meaningful and useful.
|
||||
"no-invalid-this": ["off"],
|
||||
"prefer-rest-params": ["off"]
|
||||
},
|
||||
"extends": "../../.eslintrc.json"
|
||||
|
||||
Reference in New Issue
Block a user