`npm ci` progress bar not properly removed


(Joshua) #1

What I Wanted to Do

I wanted to install with npm ci

What Happened Instead

It installed, but printed added 2 pacakges in 0.966sractTree: extractTree

Reproduction Steps

  1. npm init -y
  2. npm i debug (or any package)
  3. npm ci



npm: 6.0.1
node: 9.4.0

(on different computer than one with npm, so can’t copy paste)

(Kat Marchán) #2

this sounds like a nice thing to get a PR for. I thought I’d fixed it, but clearly not.

You “just” have to go to lib/ci.js and make sure to disable the progressbar before printing the summary, and you’re all set. :slight_smile:

In fact, you might check out whether I did it in cipm itself, 'cause I know I patched this at some point :thinking:

(Joshua) #3

Made a PR: https://github.com/npm/npm/pull/20715

(Kat Marchán) #4