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

Changes from 1 commit
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
7 changes: 6 additions & 1 deletion package.json
View file
Open in desktop
Expand Up @@ -14,7 +14,11 @@
"node": ">=12"
},
"scripts": {
"test": "xo && nyc ava && markdownlint '**/*.md'",
"lint": "npm-run-all --continue-on-error --aggregate-output --parallel lint:*",
fisker marked this conversation as resolved.
Show resolved Hide resolved
"lint:js": "xo",
"lint:md": "markdownlint '**/*.md'",
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I believe only double quote is supported on Windows.

Copy link
Sponsor Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed.

"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 +77,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