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

Make npm logs less verbose in CI #37566

Merged
merged 1 commit into from Nov 6, 2017

Conversation

Projects
None yet
2 participants
@felixfbecker
Contributor

felixfbecker commented Nov 2, 2017

This configures npm to not dump the full dependency tree for every npm install, but only the flat direct dependencies. It's impossible to fully disable the output in npm 4 (it's not printed by npm 5).

Before npm install caused >5000 log lines, which made Travis hang for multiple seconds while parsing the log in the browser. Now it's ~800, which makes the logs load significantly faster and makes Travis not hang.

Before:

image

After:

image

This does not change any install behaviour.

@joaomoreno joaomoreno merged commit 99a0ac7 into Microsoft:master Nov 6, 2017

3 checks passed

continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
license/cla All CLA requirements met.
@joaomoreno

This comment has been minimized.

Show comment
Hide comment
@joaomoreno

joaomoreno Nov 6, 2017

Member

Excellent stuff!

Member

joaomoreno commented Nov 6, 2017

Excellent stuff!

@joaomoreno joaomoreno added this to the November 2017 milestone Nov 6, 2017

@felixfbecker felixfbecker deleted the felixfbecker:quiet-npm branch Nov 6, 2017

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