Chore: [AEA-0000] - add ignore scripts option#655
Conversation
There was a problem hiding this comment.
Pull request overview
Note
Copilot was unable to run its full agentic suite in this review.
Updates the project’s Node.js dependency installation step to prevent npm lifecycle scripts from running during make install, aligning with the goal of adding an “ignore scripts” option to npm ci.
Changes:
- Add
--ignore-scriptsto thenpm cicommand in the Makefile.
|
|
||
| install-node: | ||
| npm ci | ||
| npm ci --ignore-scripts |
There was a problem hiding this comment.
npm ci --ignore-scripts will skip lifecycle scripts (e.g., postinstall) that some dependencies rely on to build native modules or generate required artifacts. If those scripts are needed for local development or CI, consider making this behavior configurable (e.g., IGNORE_SCRIPTS ?= 1 and conditionally append --ignore-scripts, or provide a separate target like install-node-no-scripts).
|



Summary
Details