gabrielcsapo / gabrielcsapo/ponticus
Clean up eslint configuration
- Dominant language
- JavaScript
- Stars
- 1
- Forks
- 0
- PR merge metrics
- No merged PRs in 30d
Description
tl;dr: we can have 1 `.estlinrc.json` file for the whole project.
ESlint only runs on JS files by default, and requires `--ext .ts` to run on typescript. If ESlint can't find any files to run, it errors out (unless a [--no-error-on-unmatched-pattern](https://eslint.org/docs/latest/user-guide/command-line-interface#--no-error-on-unmatched-pattern) is used), which causes our [CI to fail](https://github.com/gabrielcsapo/ponticus/actions/runs/3772334516/jobs/6413203283).
This can be fixed by passing `--ext .ts` (or `--ext .ts,.js` for packages that have both JS and TS) in the package.json lint script.
We should create a `.eslintrc.json` at the root level and take advantage of [the cascade](https://eslint.org/docs/latest/user-guide/configuring/configuration-files#cascading-and-hierarchy) to simplify the configuration.
Contributor guide
No contributing guide indexed for this repository
Assessment
This issue has not been assessed yet.