Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Use npm-run-all to run test and lint scripts #1604

Merged
merged 8 commits into from Nov 12, 2021
Merged

Use npm-run-all to run test and lint scripts #1604

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
810e9d4
test: use npm-run-all to run test and lint scripts
bmish Nov 11, 2021
10d1a2d
double quote
bmish Nov 12, 2021
a12e64b
run-p, add fix scripts
bmish Nov 12, 2021
8e7e6d2
continue on error
bmish Nov 12, 2021
8c80e27
remove unnecessary quotes
bmish Nov 12, 2021
48e0589
switch to fix prefix scripts
bmish Nov 12, 2021
50fab07
use npm scripts in ci
bmish Nov 12, 2021
601d0f1
switch CI back to npx ava
bmish Nov 12, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
2 changes: 1 addition & 1 deletion .github/workflows/main.yml
View file
Open in desktop
Expand Up @@ -34,7 +34,7 @@ jobs:
with:
node-version: 16
- run: npm install
- run: npx xo
- run: npm run lint
# Force update snapshots, https://github.com/avajs/ava/discussions/2754
- run: npx nyc ava --update-snapshots
env:
Expand Down
10 changes: 9 additions & 1 deletion package.json
View file
Open in desktop
Expand Up @@ -14,7 +14,14 @@
"node": ">=12"
},
"scripts": {
"test": "xo && nyc ava && markdownlint '**/*.md'",
"fix": "run-p --continue-on-error fix:*",
"fix:js": "npm run lint:js -- --fix",
"fix:md": "npm run lint:md -- --fix",
"lint": "run-p --continue-on-error lint:*",
"lint:js": "xo",
"lint:md": "markdownlint \"**/*.md\"",
"test": "npm-run-all --continue-on-error lint test:*",
"test:js": "nyc ava",
"create-rule": "node ./scripts/create-rule.mjs && npm run generate-rules-table && npm run generate-usage-example",
"run-rules-on-codebase": "node ./test/run-rules-on-codebase/lint.mjs",
"integration": "node ./test/integration/test.mjs",
Expand Down Expand Up @@ -73,6 +80,7 @@
"lodash-es": "4.17.21",
"markdownlint-cli": "^0.29.0",
"mem": "^9.0.1",
"npm-run-all": "^4.1.5",
"nyc": "^15.1.0",
"outdent": "^0.8.0",
"typescript": "^4.4.2",
Expand Down