@aneuhold/eslint-config 2.0.2 → 2.0.4
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/package.json +1 -1
- package/src/svelte-config.js +21 -9
- package/src/ts-lib-config.js +3 -0
package/package.json
CHANGED
package/src/svelte-config.js
CHANGED
|
@@ -111,6 +111,25 @@ export default defineConfig(
|
|
|
111
111
|
// Turned off because it doesn't seem too helpful, and it likes to error
|
|
112
112
|
// on things that seem to be just fine in generics.
|
|
113
113
|
'@typescript-eslint/no-unnecessary-type-parameters': 'off',
|
|
114
|
+
// Ban type assertions (as X) to prevent type-avoidance
|
|
115
|
+
'@typescript-eslint/consistent-type-assertions': ['error', { assertionStyle: 'never' }],
|
|
116
|
+
// Ban the `object` type to prevent type-avoidance
|
|
117
|
+
'@typescript-eslint/no-restricted-types': [
|
|
118
|
+
'error',
|
|
119
|
+
{
|
|
120
|
+
types: {
|
|
121
|
+
object: {
|
|
122
|
+
message: 'The `object` type is too loose. Use a specific type instead.',
|
|
123
|
+
},
|
|
124
|
+
},
|
|
125
|
+
},
|
|
126
|
+
],
|
|
127
|
+
// Disabled due to false positives with Svelte components
|
|
128
|
+
'@typescript-eslint/no-useless-default-assignment': 'off',
|
|
129
|
+
// Disabled because it wasn't working correctly with Svelte snippets
|
|
130
|
+
'@typescript-eslint/no-confusing-void-expression': 'off',
|
|
131
|
+
// Disabled because it doesn't make sense IMO. This should be disabled globally.
|
|
132
|
+
'@typescript-eslint/prefer-literal-enum-member': 'off',
|
|
114
133
|
},
|
|
115
134
|
},
|
|
116
135
|
|
|
@@ -148,15 +167,8 @@ export default defineConfig(
|
|
|
148
167
|
{
|
|
149
168
|
files: ['**/*.svelte'],
|
|
150
169
|
rules: {
|
|
151
|
-
|
|
152
|
-
|
|
153
|
-
{
|
|
154
|
-
ignoreGoto: false,
|
|
155
|
-
ignoreLinks: true,
|
|
156
|
-
ignorePushState: false,
|
|
157
|
-
ignoreReplaceState: false,
|
|
158
|
-
},
|
|
159
|
-
],
|
|
170
|
+
// Disabling this because we have a bunch of dynamic routes
|
|
171
|
+
'svelte/no-navigation-without-resolve': 'off',
|
|
160
172
|
},
|
|
161
173
|
},
|
|
162
174
|
|
package/src/ts-lib-config.js
CHANGED
|
@@ -27,6 +27,9 @@ const defaultConfig = defineConfig(
|
|
|
27
27
|
// Makes it so that there's 1 line above tags in jsdoc comments.
|
|
28
28
|
'jsdoc/tag-lines': ['error', 'any', { startLines: 1 }],
|
|
29
29
|
'jsdoc/require-returns': 'off',
|
|
30
|
+
// Disabled because it doesn't seem to support triple back-tick code blocks at the moment
|
|
31
|
+
// 2/21/2026.
|
|
32
|
+
'jsdoc/escape-inline-tags': 'off',
|
|
30
33
|
'no-use-before-define': 'off',
|
|
31
34
|
'no-undef': 'off',
|
|
32
35
|
// Just 100% disagree with this rule. The reasoning is that using a
|