[IMP] web: configure prefer-const eslint rule to be more lax

We recently added the prefer-const eslint rule, but the default
configuration of this rule disallows some things that we often do and
which we do not think are bad practice such as declaring a variable with
let so that it's available to a function defined below, but assign it
only once later, we also sometimes destructure variables with let where
some of the variables could be const but not all, in those cases we
accept that some variables will be mutable even though it's not strictly
necessary in favour of conciseness.

closes odoo/odoo#97384

Signed-off-by: Lucas Perais (lpe) <lpe@odoo.com>
This commit is contained in:
Samuel Degueldre
2022-08-03 15:10:04 +02:00
parent 2542e91fb7
commit 1107e18234
+4 -1
View File
@@ -21,7 +21,10 @@
"valid-typeof": ["error"],
"no-unused-vars": ["error", { "vars": "all", "args": "none", "ignoreRestSiblings": false, "caughtErrors": "all", "caughtErrorsIgnorePattern": "^_" }],
"curly": ["error", "all"],
"prefer-const": ["error"]
"prefer-const": ["error", {
"destructuring": "all",
"ignoreReadBeforeAssign": true
}]
},
"globals": {
"owl": "readonly",