@feltjs/eslint-config

an ESLint config for TypeScript, Svelte, and Prettier

npm i -D @feltjs/eslint-config

This ESLint config is designed to complement TypeScript and Prettier, so all redundant rules are disabled. It extends no other configs, so wysiwyg👀

To distinguish linting problems from type errors while editing, all of the rules are set to warn, not error, and users are expected run eslint --max-warnings 0 to treat them as errors when desired. There may be a better way to do this, but it works well to get yellow squigglies in text editors for linting problems while reserving red for type errors.

Got questions or proposed changes? Open issues!

setup

Install ESLint dev dependencies with TypeScript and Svelte plugins:

npm i -D eslint \
	@typescript-eslint/eslint-plugin @typescript-eslint/parser \
	eslint-plugin-svelte \
	@feltjs/eslint-config

Then add the following to your project's package.json or an ESLint config file:

{
	// ...package.json stuff
	"eslintConfig": {
		"root": true,
		"extends": "@feltjs"
	}
}

vscode setup

To make the VSCode ESLint plugin work with Svelte, add the following to VSCode's settings.json:

{
	// ctrl+shift+P then `> Preferences: Open Settings (JSON)`
	// ...other config
	"eslint.validate": ["svelte", "javascript", "javascriptreact"]
}

See also the eslint-plugin-svelte integrations docs.

overrides

Every ESLint config has opinions. This project is somewhat strict, because it's easier to disable rules than it is to enable ones that are invisibly disabled, but we don't want it to be too opinionated. We're happy to discuss proposed changes in the issues, and to override any defaults, use the rules property in your config:

{
	"eslintConfig": {
		"root": true,
		"extends": "@feltjs",
		"rules": {
			"no-param-reassign": 0, // disable an enabled rule
			"no-console": 1 // enable a disabled rule
		}
	}
}

features

One useful opt-in feature is implemented with the following:

'no-warning-comments': [1, {terms: ['todo block']}],

This can be used to leave notes to yourself while you develop a set of changes, and combined with CI, you'll be blocked from merging until they're all resolved:

// TODO BLOCK
// TODO BLOCK do the thing
// todo block is case insensitive

develop

See index.cjs and the ESLint docs.

Rules are current through eslint@8.20.0 and @typescript-eslint/eslint-plugin@5.30.7.

publish

This repo uses changesets:

changeset version
changeset publish
git push
git push --tags

See the official docs for npm version and npm publish.

credits 🐢_🐢🐢

ESLint ∙ @typescript-eslint/eslint-plugin ∙ eslint-plugin-svelte ∙ TypeScript ∙ Svelte ∙ Prettier & more

license 🐦

public domain ⚘ The Unlicense