Skip to content

Commit

Permalink
Use the npm lint script on CI.
Browse files Browse the repository at this point in the history
This should be more consistent since it should use the specified standard version and not the latest one.

PR-URL: #604
Credit: @XhmikosR
Close: #604
Reviewed-by: @mikemimik
  • Loading branch information
XhmikosR authored and Michael Perrotte committed Jan 23, 2020
1 parent b1aeeb6 commit 7852c0c
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 2 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,7 @@ jobs:
# run: sudo PATH=$PATH $(which node) . run tap -- "test/tap/*.js" --coverage --timeout 600

- name: Lint
run: npx standard
run: node . run lint

- name: Validate licenses
run: node . run licenses
3 changes: 2 additions & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -296,7 +296,8 @@
"licenses": "licensee --production --errors-only",
"tap": "tap -J --timeout 300 --no-esm",
"tap-cover": "tap -J --nyc-arg=--cache --coverage --timeout 600 --no-esm",
"pretest": "standard",
"lint": "standard",
"pretest": "npm run lint",
"test": "npm run test-tap --",
"test:nocleanup": "NO_TEST_CLEANUP=1 npm run test --",
"sudotest": "sudo npm run tap -- \"test/tap/*.js\"",
Expand Down

0 comments on commit 7852c0c

Please sign in to comment.