-
Notifications
You must be signed in to change notification settings - Fork 3.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[BUG] v7.3.0 publish --dry-run
will require login, and publishConfig
from package.json
load after the check (ENEEDAUTH)
#2411
Comments
Can confirm this is happening on my end also. Additionally,
Environment
|
Upon further inspection, this seems related to #2360 and changes in be4a090. In |
Thanks @foxxyz, that workaround worked for me too. I experienced this issue when attempting to deploy to a GitLab private repo. I would imagine this would break the build pipeline for anyone else following the example configuration in the GitLab docs: https://docs.gitlab.com/ee/user/packages/npm_registry/#publish-an-npm-package-by-using-cicd |
publish --dry-run
will require login (ENEEDAUTH)publish --dry-run
will require login, and publishConfig
from package.json
load after the check (ENEEDAUTH)
I don't want to sidetrack this issue too much but if |
@thw0rted try |
Nice! Thanks, I just learned about the command today but didn't realize it was actually making a network request. Adding a One caveat is that |
notable change: - deprecate: `fileProcessorWebpack` - add: `copyAfterEdit` to `node/file` - temp-fix: ci: wait for npm/cli#2411 - script sort - package update
notable change: - deprecate: `fileProcessorWebpack` - add: `copyAfterEdit` to `node/file` - temp-fix: ci: wait for npm/cli#2411 - script sort - package update
notable change: - deprecate: `fileProcessorWebpack` - temp-fix: ci: wait for npm/cli#2411 - temp-fix: ci: cache: wait for: actions/runner-images#2153 - fix: use `@dr-js/core@0.4.3-dev.1` with fix for nodejs@15.5 caused responder send noop - fix: output: log git diff in git status check - fix: test: treat `before|after` the same as `it` - add: `copyAfterEdit` to `node/file` - add: bin: `is-hash-changed` to `cache-step` - add: ci: `.github/ci-patch.js` for doing ci patch in one place - node: use `getToggle` for `Preset.Toggle` - bin: suppot `checksum-file-only` for `cache-step` - bin: sort babel resolve path & test - bin: sort `cache-step` options - ci & script sort - package update
Current Behavior:
publish --dry-run
with v7.3.0 will require login (ENEEDAUTH)Expected Behavior:
publish --dry-run
should work when not login, like in CI env.Steps To Reproduce:
Sample error log:
Environment:
The text was updated successfully, but these errors were encountered: