mirror of
https://github.com/overleaf/overleaf.git
synced 2024-11-21 20:47:08 -05:00
12890edd14
[web] fix eslint violations for node/no-callback-literal GitOrigin-RevId: 8d6bb1398b3db2794bf1b2f97cd6d2886f2b4b0a
183 lines
4.9 KiB
Text
183 lines
4.9 KiB
Text
{
|
|
"root": true,
|
|
"extends": [
|
|
"eslint:recommended",
|
|
"standard",
|
|
"prettier"
|
|
],
|
|
"env": {
|
|
"es2020": true
|
|
},
|
|
"settings": {
|
|
// Tell eslint-plugin-react to detect which version of React we are using
|
|
"react": {
|
|
"version": "detect"
|
|
}
|
|
},
|
|
"rules": {
|
|
// Disable some rules after upgrading ESLint
|
|
// TODO: re-enable and fix
|
|
"no-var": "off",
|
|
|
|
// do not allow importing of implicit dependencies.
|
|
"import/no-extraneous-dependencies": "error",
|
|
|
|
"node/no-deprecated-api": "off"
|
|
},
|
|
"overrides": [
|
|
// NOTE: changing paths may require updating them in the Makefile too.
|
|
{
|
|
// Node
|
|
"files": [
|
|
"**/app/src/**/*.js",
|
|
"app.js",
|
|
"i18next-scanner.config.js",
|
|
"karma.conf.js",
|
|
"scripts/**/*.js",
|
|
"webpack.config*.js"
|
|
],
|
|
"env": {
|
|
"node": true
|
|
}
|
|
},
|
|
{
|
|
// Test specific rules
|
|
"files": ["**/test/**/*.js"],
|
|
"plugins": [
|
|
"mocha",
|
|
"chai-expect",
|
|
"chai-friendly"
|
|
],
|
|
"env": {
|
|
"mocha": true
|
|
},
|
|
"rules": {
|
|
// mocha-specific rules
|
|
"mocha/handle-done-callback": "error",
|
|
"mocha/no-exclusive-tests": "error",
|
|
"mocha/no-global-tests": "error",
|
|
"mocha/no-identical-title": "error",
|
|
"mocha/no-nested-tests": "error",
|
|
"mocha/no-pending-tests": "error",
|
|
"mocha/no-skipped-tests": "error",
|
|
"mocha/no-mocha-arrows": "error",
|
|
|
|
|
|
// Swap the no-unused-expressions rule with a more chai-friendly one
|
|
"no-unused-expressions": "off",
|
|
"chai-friendly/no-unused-expressions": "error",
|
|
|
|
// chai-specific rules
|
|
"chai-expect/missing-assertion": "error",
|
|
"chai-expect/terminating-properties": "error",
|
|
|
|
// prefer-arrow-callback applies to all callbacks, not just ones in mocha tests.
|
|
// we don't enforce this at the top-level - just in tests to manage `this` scope
|
|
// based on mocha's context mechanism
|
|
"mocha/prefer-arrow-callback": "error"
|
|
}
|
|
},
|
|
{
|
|
// Frontend test specific rules
|
|
"files": ["**/test/karma/**/*.js"],
|
|
"globals": {
|
|
"expect": true,
|
|
"$": true
|
|
}
|
|
},
|
|
{
|
|
// Backend specific rules
|
|
"files": ["**/app/src/**/*.js", "app.js"],
|
|
"rules": {
|
|
// don't allow console.log in backend code
|
|
"no-console": "error",
|
|
|
|
// do not allow importing of implicit dependencies.
|
|
"import/no-extraneous-dependencies": ["error", {
|
|
// do not allow importing of devDependencies.
|
|
"devDependencies": false
|
|
}]
|
|
}
|
|
},
|
|
{
|
|
// Frontend specific rules
|
|
"files": ["**/frontend/js/**/*.js", "**/frontend/stories/**/*.js", "**/*.stories.js", "**/test/frontend/**/*.js"],
|
|
"env": {
|
|
"browser": true
|
|
},
|
|
"parserOptions": {
|
|
"sourceType": "module",
|
|
"ecmaFeatures": {
|
|
"jsx": true
|
|
}
|
|
},
|
|
"plugins": [
|
|
"jsx-a11y"
|
|
],
|
|
"extends": [
|
|
"plugin:react/recommended",
|
|
"plugin:react-hooks/recommended",
|
|
"plugin:jsx-a11y/recommended",
|
|
"standard-jsx",
|
|
"standard-react",
|
|
"prettier"
|
|
],
|
|
"globals": {
|
|
"__webpack_public_path__": true,
|
|
"$": true,
|
|
"angular": true,
|
|
"ace": true,
|
|
"ga": true,
|
|
"sl_console": true,
|
|
"sl_debugging": true,
|
|
// Injected in layout.pug
|
|
"user_id": true,
|
|
"ExposedSettings": true
|
|
},
|
|
"rules": {
|
|
"react/jsx-no-target-blank": ["error", {
|
|
"allowReferrer": true
|
|
}],
|
|
// Prevent usage of legacy string refs
|
|
"react/no-string-refs": "error",
|
|
|
|
// Prevent curly braces around strings (as they're unnecessary)
|
|
"react/jsx-curly-brace-presence": ["error", {
|
|
"props": "never",
|
|
"children": "never"
|
|
}],
|
|
|
|
// Don't import React for JSX; the JSX runtime is added by a Babel plugin
|
|
"react/react-in-jsx-scope": "off",
|
|
"react/jsx-uses-react": "off",
|
|
|
|
// Fix conflict between prettier & standard by overriding to prefer
|
|
// double quotes
|
|
"jsx-quotes": ["error", "prefer-double"],
|
|
|
|
// Override weird behaviour of jsx-a11y label-has-for (says labels must be
|
|
// nested *and* have for/id attributes)
|
|
"jsx-a11y/label-has-for": [
|
|
"error",
|
|
{
|
|
"required": {
|
|
"some": [
|
|
"nesting",
|
|
"id"
|
|
]
|
|
}
|
|
}
|
|
]
|
|
}
|
|
},
|
|
{
|
|
"files": ["scripts/ukamf/*.js"],
|
|
"rules": {
|
|
// Do not allow importing of any dependencies unless specified in either
|
|
// - web/package.json
|
|
// - web/scripts/ukamf/package.json
|
|
"import/no-extraneous-dependencies": ["error", {"packageDir": [".", "scripts/ukamf"]}]
|
|
}
|
|
}
|
|
]
|
|
}
|