(eslint): update config to handle ES modules and Odoo JSDoc tags
Merged
(eslint): update config to handle ES modules and Odoo JSDoc tags
topic/18.0/ob
into
branch/18.0
2 unresolved threads
2 unresolved threads
Merge request reports
Activity
Filter activity
added 2 commits
- 8a2774a95da5 - (eslint): update config to handle ES modules and Odoo JSDoc tags
- 70a867936d61 - /** @odoo-module **/ can be removed from the JS files.
added 2 commits
- 023a00eec270 - (eslint): update config to handle ES modules and Odoo JSDoc tags
- 7a326596c427 - /** @odoo-module **/ can be removed from the JS files.
added 2 commits
- 13cbbc387b6f - (eslint): update config to handle ES modules and Odoo JSDoc tags
- ba7e67cc4e02 - /** @odoo-module **/ can be removed from the JS files.
assigned to @vincent.hatakeyama
194 196 }, 195 197 }, 196 198 { 197 files: ["**/*.esm.js"], 199 files: ["**/*.esm.js", "**/*.js"], -
C’est volontaire que les fichiers doivent finir en .esm.js pour que les règles suivantes s’appliquent. Je pense qu’il faut renommer
static/src/components/dynamic_placeholder_char_field/dynamic_placeholder_char_field.jsà la place. changed this line in version 5 of the diff
17 17 openerp: "readonly", 18 18 owl: "readonly", 19 19 luxon: "readonly", 20 InputEvent: "readonly", 21 KeyboardEvent: "readonly", - Comment on lines +20 to +21
En changeant ces lignes, la configuration ne correspond plus à celle d’OCA (eslint.config.cjs).
Est-ce nécessaire et est-ce que la configuration doit être appliquée pour tous les modules Odoo ?
assigned to @oury.balde and unassigned @vincent.hatakeyama
added 6 commits
-
ba7e67cc...bf4e5463 - 4 commits from branch
branch/18.0 -
81d56a814d2f -
- 24827a60a72f - (eslint): update config to handle ES modules and Odoo JSDoc tags
-
ba7e67cc...bf4e5463 - 4 commits from branch
Please register or sign in to reply