Use eslint 9 to lint js and ts repos with arc lint. Specify a config file that supports current ruleset.
Note that some js/ts repos are not currently covered by .arclint. Will add these in subsequent diffs (ecash-lib, ecash-agora)
Details
Details
- Reviewers
- None
- Group Reviewers
Restricted Project
arc lint --everything
Diff Detail
Diff Detail
- Repository
- rABC Bitcoin ABC
- Branch
- eslint-to-9
- Lint
Lint Passed - Unit
No Test Coverage - Build Status
Buildable 32576 Build 64640: Build Diff ecash-herald-tests Build 64639: arc lint + arc unit
Event Timeline
| .arclint | ||
|---|---|---|
| 314 | arguably the removal of alias-server is unrelated to this diff, but we are doing some other generalized improvements in line with the eslint upgrade (including js files now that we can globally configure eslint to lint them properly) | |
| apps/ecash-herald/tsconfig.json | ||
| 11 | this is definitely unrelated but once this was fixed with arc lint I was not able to back it out. I think it's ok to let this change stay in this diff given that it was a lint autofix | |
| arcanist/linter/ESLinter.php | ||
| 39 | this caused issues users would have to have eslint globally installed, and it would have to be the correct version now the version is pinned top level of the monorepo and this is the binary used for linting the js files. | |