Official Neo Financial ESLint configuration
This package includes 4 different ESLint configs:
config-backendconfig-frontendconfig-backend-nextconfig-frontend-next
The next versions include some rules that are being considered for inclusion in future versions of the base config. The next configs also require you to specify the project setting in parserOptions for TypeScript projects. The will make ESLint run slower in TypeScript projects.
flowchart LR
A[config-base]
B[config-base-next]
C[config-backend]
D[config-frontend]
E[config-backend-next]
F[config-frontend-next]
A --> C
A --> D
A --> B
B --> E
B --> F
The arrows from left to right illustrate which configs are extended by another config.
npm install --save-dev eslint-config-neo
You can also install a specific version of the package by appending the version tag. For example, to install version 1.0.0
npm install --save-dev [email protected]
npm install -D eslint prettier lint-staged husky typescriptAdd .eslintrc to project root
{
"extends": "eslint-config-neo/config-backend"
}Use eslint-config-neo/config-frontend for frontend projects
If you're using one of the next configs you must set the project option to include all of your tsconfig.json files:
{
"extends": "eslint-config-neo/config-backend",
"parserOptions": {
"project": ["tsconfig.json", "test/tsconfig.json"]
}
}Add .prettierrc to project root
{
"printWidth": 120,
"singleQuote": true
}Optional: If there are any files you want to exclude from Prettier add .prettierignore to project root
Add .editorconfig to project root
# http://editorconfig.org
root = true
[*]
charset = utf-8
end_of_line = lf
indent_size = 2
indent_style = space
insert_final_newline = true
max_line_length = 120
[*.md]
max_line_length = 0
trim_trailing_whitespace = false
[COMMIT_EDITMSG]
max_line_length = 0Add the engines field to package.json
"engines": {
"node": "^12.0.0"
}Add scripts for linting and formatting to package.json
"scripts": {
"lint": "eslint .",
"format": "prettier --write \"**/*.{ts,tsx,js,json,graphql,md}\"",
"format:check": "prettier --debug-check \"**/*.{ts,tsx,js,json,graphql,md}\""
}Add a precommit hook to package.json to automatically lint and format any files staged for commit
"husky": {
"hooks": {
"pre-commit": "lint-staged"
}
},
"lint-staged": {
"concurrent": false,
"linters": {
"*.{ts,tsx,js}": [
"eslint --quiet",
"git add"
],
"*.{ts,tsx,js,json,graphql,md}": [
"prettier --write",
"git add"
]
}
}If you've added Prettier to an existing project you will want to format all the code. The precommit hook only updates files that have been changed and staged for commit. To format the entire codebase run
npm run format
- requires
eslintversion8.26.0or higher - for backend services, you should delete the domain-specific
eslintrc
Should I override the rule X? I don't like it.
No. If you want to do this, you have to ask the team. If a rule really doesn't make sense then we should remove or disable it.
Can I disable the rule on one line in my code, I have a good reason.
Yes. That's ok. Use a single-line disable.
- Update the version in
package.json - Create a
CHANGELOGentry - Commit your changes
npm pack --dry-runto see what will be publishednpm publish- Create a release on GitHub. Use the version as the tag and release name. For example for version
1.0.0the tag and release name would bev1.0.0. Add theCHANGELOGdetails to the release.
- Update the version in
package.jsonto be postfixed with-next.xwherexis a number
- for example your first test version might look like
0.7.1-next.0, second test version would havenext.1, etc
- run
npm publish --canary --exact --preid canary --tag=canary - In the project you wish to test in, run
npm i eslint-config-neo@canary