apm icon indicating copy to clipboard operation
apm copied to clipboard

apm install --silent parameter isn't respected

Open lee-dohm opened this issue 10 years ago • 3 comments

$ apm install clippy
Installing clippy to /Volumes/Data/Users/Lee/.atom/packages ✓
$ apm install --quiet clippy
Installing clippy to /Volumes/Data/Users/Lee/.atom/packages ✓
$ apm install --silent clippy
Installing clippy to /Volumes/Data/Users/Lee/.atom/packages ✓

Perhaps --quiet is respected, but I would expect --silent to not give any output, just a return code.

lee-dohm avatar May 16 '14 16:05 lee-dohm

Looking at this issue again, I'm not sure if this is a bug. Here's how the expected behavior of he --silent flag is described:

$ apm install --help

Usage: apm install [<package_name>...]
...

Options:
  ...
  --silent, -s      Set the npm log level to silent                              [boolean]
  --quiet, -q       Set the npm log level to warn                                [boolean]

My interpretation of this is that silence isn't promised for apm, but rather apm promises to pass the flag along to npm, which it does.

@lee-dohm Given this, do you think the output matches the documented behavior? I guess we could turn this into a feature request so that apm supports --quiet and --silent as well? /cc @kevinsawicki

izuzak avatar Aug 23 '14 15:08 izuzak

Yes, I think it does match the documented behavior. Though, I still feel the expectation is for --silent to not produce any output and for --quiet to produce minimal output ... regardless of the distinction between apm and npm. I'm fine with it being a feature request :grinning:

Thanks for following up!

lee-dohm avatar Aug 23 '14 17:08 lee-dohm

I'd like this feature.

ghost avatar Oct 23 '17 06:10 ghost