fix(libnpmpack): obey ignoreScripts

[winterqt]

Per config:

If true, npm does not run scripts specified in package.json files.

Note that commands explicitly intended to run a particular script, such as npm start, npm stop, npm restart, npm test, and npm run-script will still run their intended script if ignore-scripts is set, but they will not run any pre- or post-scripts.

References

None

[wraithgar]

Just one test nitpick. This one certainly went much more smoothly than the last one lol.

[winterqt]

Wonder why this wasn't mentioned in the changelogs, but my previous PR was. 🤔

Thought this wasn't included in v9 for a second, haha.

[wraithgar]

Yeah the changes in workspaces themselves show up in their respective changelogs. We're still looking into a good way to bubble those up to the cli w/o having to have EVERY change from EVERY dependency end up in the changelog too.

[winterqt]

So, just out of curiosity, how did the ignoreScripts change bubble up to the cli changelog? Was that done manually or something?

[wraithgar]

That change was in 5.0.0-pre.3 of libnpmpack, which was a dependency update in 9.0.0-pre.5 of npm

[wraithgar]

All of our changelog entries are automated via release-please through github actions.

#5689