mirror of
https://github.com/sveltejs/svelte.git
synced 2024-11-28 16:12:17 +01:00
ca6e9b5745
* chore: don't swallow rejected promise errors * ignore rules rather than adding catch blocks. add async to functions returning promises * remove redundant asyncs * remove extremely pointless rule * remove another daft rule * this is what typescript is for * again, typescript already has this covered, we don't need it * simplify * this rule is harmless --------- Co-authored-by: Rich Harris <rich.harris@vercel.com>
60 lines
1.7 KiB
JavaScript
60 lines
1.7 KiB
JavaScript
import svelte_config from '@sveltejs/eslint-config';
|
|
import lube from 'eslint-plugin-lube';
|
|
|
|
/** @type {import('eslint').Linter.FlatConfig[]} */
|
|
export default [
|
|
...svelte_config,
|
|
{
|
|
languageOptions: {
|
|
parserOptions: {
|
|
project: true
|
|
}
|
|
},
|
|
plugins: {
|
|
lube
|
|
},
|
|
rules: {
|
|
'@typescript-eslint/await-thenable': 'error',
|
|
'@typescript-eslint/prefer-promise-reject-errors': 'error',
|
|
'@typescript-eslint/require-await': 'error',
|
|
'no-console': 'error',
|
|
'lube/svelte-naming-convention': ['error', { fixSameNames: true }],
|
|
// eslint isn't that well-versed with JSDoc to know that `foo: /** @type{..} */ (foo)` isn't a violation of this rule, so turn it off
|
|
'object-shorthand': 'off',
|
|
'no-var': 'off',
|
|
|
|
// TODO: enable these rules and run `pnpm lint:fix`
|
|
// skipping that for now so as to avoid impacting real work
|
|
'@stylistic/quotes': 'off',
|
|
'@typescript-eslint/no-unused-vars': 'off',
|
|
'prefer-const': 'off'
|
|
}
|
|
},
|
|
{
|
|
files: ['playgrounds/**/*'],
|
|
rules: {
|
|
'lube/svelte-naming-convention': 'off',
|
|
'no-console': 'off'
|
|
}
|
|
},
|
|
{
|
|
ignores: [
|
|
'**/*.d.ts',
|
|
'**/tests',
|
|
'packages/svelte/scripts/process-messages/templates/*.js',
|
|
'packages/svelte/src/compiler/errors.js',
|
|
'packages/svelte/src/internal/client/errors.js',
|
|
'packages/svelte/src/internal/client/warnings.js',
|
|
'packages/svelte/src/internal/shared/warnings.js',
|
|
'packages/svelte/compiler/index.js',
|
|
// documentation can contain invalid examples
|
|
'documentation',
|
|
// contains a fork of the REPL which doesn't adhere to eslint rules
|
|
'sites/svelte-5-preview/**',
|
|
'tmp/**',
|
|
// wasn't checked previously, reenable at some point
|
|
'sites/svelte.dev/**'
|
|
]
|
|
}
|
|
];
|