• Home
  • Features
  • Pricing
  • Docs
  • Announcements
  • Sign In

npm / cli / #1185
98%
latest: 97%

Build:
Build:
LAST BUILD BRANCH: nlf/arborist-validation
DEFAULT BRANCH: latest
Ran 02 Feb 2021 10:56PM UTC
Jobs 1
Files 120
Run time 179min
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

pending completion
#1185

push

wraithgar
fix(publish): follow config for registry auth check

The "do you have auth configured" only takes into consideration a hard-coded
"registry" config, which means that if you don't have auth configured for
the npm registry, but you do for the one you have tied to a scope elsewhere
in your npmrc, we would erroneously tell you that to add a token.

This uses the same method that the rest of the publish chain uses to determine
which registry it would be publishing to, then sees if you have auth for THAT registry.

Because that other function does a hard fallback to the npm registry, there is no more
need for the exception we throw if you do not have one configured.  Also, the npm cli
already defaults that config item, so you can't even set it to a falsey value if you
wanted

2809 of 2985 branches covered (94.1%)

Branch coverage included in aggregate %.

4967 of 5198 relevant lines covered (95.56%)

17.3 hits per line

Jobs
ID Job ID Ran Files Coverage
1 #1185.1 02 Feb 2021 10:56PM UTC 0
95.03
Source Files on build #1185
Detailed source file information is not available for this build.
  • Back to Repo
  • a8e61556 on github
  • Prev Build on gar/alt-registry-auth
  • Next Build on gar/alt-registry-auth
STATUS · Troubleshooting · Open an Issue · Sales · Support · CAREERS · ENTERPRISE · START FREE · SCHEDULE DEMO
ANNOUNCEMENTS · TWITTER · TOS & SLA · Supported CI Services · What's a CI service? · Automated Testing

© 2026 Coveralls, Inc