Skip to content

Commit

Permalink
Move lint script to sitting under npm-run-all, to allow parallel ru…
Browse files Browse the repository at this point in the history
…nners (#517)

* Move lint script to sitting under `npm-run-all`, to allow parallel runners

* Ensure the GitHub Action runners uses the combined linter to cut down on its runtime as well.
  • Loading branch information
Clorith authored Aug 21, 2024
1 parent 82b32cc commit 9c1d168
Show file tree
Hide file tree
Showing 3 changed files with 236 additions and 6 deletions.
7 changes: 2 additions & 5 deletions .github/workflows/lint-js-css.yml
Original file line number Diff line number Diff line change
Expand Up @@ -34,8 +34,5 @@ jobs:
- name: Install Dependencies
run: npm ci

- name: Lint CSS
run: npm run lint:css

- name: Lint JS
run: npm run lint:js
- name: Run linters
run: npm run lint
232 changes: 232 additions & 0 deletions package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

3 changes: 2 additions & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -25,6 +25,7 @@
"eslint-plugin-prettier": "^5.1.3",
"husky": "^9.0.11",
"lint-staged": "^15.2.7",
"npm-run-all": "^4.1.5",
"postcss": "^8.4.38",
"postcss-custom-media": "^10.0.6",
"postcss-discard-comments": "^7.0.1",
Expand All @@ -47,7 +48,7 @@
"clean": "turbo run clean && rm -rf node_modules",
"format": "prettier --write .",
"check-engines": "wp-scripts check-engines",
"lint": "npm run lint:js && npm run lint:css && npm run lint:format",
"lint": "npm-run-all --parallel lint:js lint:css lint:format",
"lint:css": "wp-scripts lint-style",
"lint:js": "wp-scripts lint-js",
"lint:format": "prettier --check .",
Expand Down

0 comments on commit 9c1d168

Please sign in to comment.