Skip to content

Commit dec6591

Browse files
committed
Switched reportUnusedDisableDirectives to on and see if there are any warnings
1 parent c0987e3 commit dec6591

1 file changed

Lines changed: 7 additions & 4 deletions

File tree

app/web/eslint.config.mjs

Lines changed: 7 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -25,14 +25,17 @@ const config = [
2525
"proto/**",
2626
],
2727
},
28-
// eslint 9 flipped this default to "warn"; keep the 8.x behavior so stale
29-
// disable directives in the tree don't fail --max-warnings=0
28+
// eslint 9 flipped this default to "warn";
3029
{
31-
linterOptions: { reportUnusedDisableDirectives: "off" },
30+
linterOptions: { reportUnusedDisableDirectives: "on" },
3231
},
32+
33+
3334
// both legacy extends must go through the same FlatCompat instance so the
3435
// @typescript-eslint plugin resolves to one module object ("Cannot redefine
3536
// plugin" otherwise)
37+
38+
// Come back to this after we upgrade next.js https://github.com/Couchers-org/couchers/issues/9280
3639
...compat.extends(
3740
"plugin:@typescript-eslint/recommended",
3841
"next/core-web-vitals",
@@ -75,7 +78,7 @@ const config = [
7578
"@typescript-eslint/no-empty-function": "off",
7679
//not using this right now
7780
"@next/next/no-img-element": "off",
78-
//this is ugly
81+
7982
"react/no-unescaped-entities": "off",
8083
// Prefer inferred types so that the code is as close to JS as possible
8184
"@typescript-eslint/explicit-module-boundary-types": "off",

0 commit comments

Comments
 (0)