Skip to content
This repository has been archived by the owner. It is now read-only.

ci: pause log before logging summary #20715

Merged
merged 2 commits into from May 23, 2018
Merged

ci: pause log before logging summary #20715

merged 2 commits into from May 23, 2018

Conversation

@legodude17
Copy link
Contributor

@legodude17 legodude17 commented May 22, 2018

From here, this will remove the progress bar before logging the summary, so that it doesn't log the summary over the progress bar and print something like added 2 pacakges in 0.966sractTree: extractTree.

@legodude17 legodude17 requested a review from as a code owner May 22, 2018
lib/ci.js Outdated Show resolved Hide resolved
zkat
zkat approved these changes May 23, 2018
Copy link
Contributor

@zkat zkat left a comment

Looks good!

@legodude17
Copy link
Contributor Author

@legodude17 legodude17 commented May 23, 2018

@zkat Great!

@zkat zkat changed the base branch from latest to release-next May 23, 2018
@zkat zkat merged commit e26316c into npm:release-next May 23, 2018
1 check passed
@legodude17
Copy link
Contributor Author

@legodude17 legodude17 commented May 23, 2018

🎉

@legodude17 legodude17 deleted the patch-5 branch May 23, 2018
iarna added a commit that referenced this issue May 24, 2018
iarna added a commit that referenced this issue May 24, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants