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.

Sign up for GitHub

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

Jump to bottom

Tests: ensure 'npm install' script output can be captured #181

Merged

vweevers merged 1 commit into prebuild:master from lovell:replace-npm7-temp-workaround

May 30, 2022

Member

lovell commented May 30, 2022

Replaces the temporary logfile workaround with the use of the foreground-scripts setting, available in npm v7+.

https://docs.npmjs.com/cli/v8/using-npm/config#foreground-scripts

(This should hopefully make removing npmlog a tiny bit easier.)


          Ensure script output can be captured by tests

1173edd

Replaces temporary logfile workaround

vweevers approved these changes

View reviewed changes

Member

vweevers left a comment

Nice one

vweevers merged commit d1853cb into prebuild:master

lovell deleted the replace-npm7-temp-workaround branch

May 30, 2022 19:29

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment