npm / npm / 7365

93%

master: %

push

travis-ci

config: don't ham-it-up when expecting parseable output (#16336)

* Don't ham-it-up when expecting JSON

I turned on the ham-it-up configuration setting, and then noticed that I received an error when running the npm-windows-upgrade module, because it was trying to parse the JSON from calling `npm view npm versions --json` and didn't know what to do with the easter egg text at the end.

This change disables the ham-it-up text when the json option is present.

PR-URL: https://github.com/npm/npm/pull/16336
Credit: @bdukes
Reviewed-By: @zkat

3761 of 5320 branches covered (70.7%)

7325 of 8515 relevant lines covered (86.02%)

3632.88 hits per line