-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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.
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
Add npm-run-all package to simplify running scripts #37558
Conversation
The npm-run-all package allows for an easier way to run multiple scripts from the package.json file. We are going to run a few scripts to generate api docs, so instead of joining multiple commands using && it can be simplified to creating a command for each and then running: `npm-run-all api-docs:*` Package docs: https://www.npmjs.com/package/npm-run-all
Size Change: +25 B (0%) Total Size: 1.13 MB
ℹ️ View Unchanged
|
We can also update this line with the newly added command: Line 309 in 8f90816
Now that we generate the docs for blocks based on Line 290 in 8f90816
|
Use the npm-run-all in the lint-staged command that gets run by husky on pre-commit hooks. This allows for checking if any block.json and core blocks are modified that triggers an update to the reference doc.
Good catch, the npm-run-all works there also, so just added the |
Co-authored-by: Greg Ziółkowski <grzegorz@gziolo.pl>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
💯
Description
The npm-run-all package allows for an easier way to run multiple scripts
from the package.json file. We are going to run a few scripts to
generate api docs, so instead of joining multiple commands using && it
can be simplified to creating a command for each and then running:
npm-run-all api-docs:*
Package docs:
https://www.npmjs.com/package/npm-run-all
How has this been tested?
Run:
npm run docs:build
and confirm it runs the set of scripts specified.Types of changes