test: run only oxlint, not eslint (#2909)
This commit is contained in:
12
.github/workflows/validate.yml
vendored
12
.github/workflows/validate.yml
vendored
@@ -21,18 +21,6 @@ jobs:
|
||||
with:
|
||||
name: code-coverage-report
|
||||
path: coverage/*.*
|
||||
# We are trying out oxlint for a while. Please report if you ever see lint issues that eslint catches but oxlint
|
||||
# misses. We expect the opposite (oxlint should catch more issues).
|
||||
lint-preview:
|
||||
runs-on: ubuntu-latest
|
||||
|
||||
steps:
|
||||
- uses: actions/checkout@v6
|
||||
- uses: actions/setup-node@v6
|
||||
with:
|
||||
node-version-file: '.nvmrc'
|
||||
- run: npm install
|
||||
- run: npm run oxlint
|
||||
coverage:
|
||||
runs-on: ubuntu-latest
|
||||
needs: tests
|
||||
|
||||
4
Makefile
4
Makefile
@@ -51,7 +51,9 @@ validate-no-uncommitted-package-lock-changes:
|
||||
validate:
|
||||
make validate-no-uncommitted-package-lock-changes
|
||||
npm run i18n_extract
|
||||
npm run lint -- --max-warnings 0
|
||||
# We are trying out oxlint. Now that it's been working well for a while with both oxlint and eslint, we have disabled
|
||||
# eslint, and after a few weeks we'll evaluate whether any problems are slipping through if only oxlint is used.
|
||||
npm run oxlint
|
||||
npm run types
|
||||
npm run test:ci
|
||||
npm run build
|
||||
|
||||
Reference in New Issue
Block a user