Skip to content
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] npm registry not showed on publish --dry-run #4317

Closed
2 tasks done
Eomm opened this issue Jan 24, 2022 · 4 comments · Fixed by #4726
Closed
2 tasks done

[BUG] npm registry not showed on publish --dry-run #4317

Eomm opened this issue Jan 24, 2022 · 4 comments · Fixed by #4726
Assignees
Labels
Bug thing that needs fixing Release 8.x work is associated with a specific npm 8 release

Comments

@Eomm
Copy link
Contributor

Eomm commented Jan 24, 2022

Is there an existing issue for this?

  • I have searched the existing issues

This issue exists in the latest npm version

  • I am using the latest npm

Current Behavior

Running:

npm publish --verbose --access restricted --dry-run

does not show the registry information added in #4143

Running the publish command without the --dry-run flag the registry is shown as expected.

Expected Behavior

The npm cli should show the registry information

Steps To Reproduce

Just run:

npm publish --verbose --access restricted --dry-run

Environment

  • npm: 8.3.1 <-- edit: fix version
  • Node.js: 17.4.0
  • OS Name: macos
  • System Model Name:
  • npm config:
; "user" config from /Users/mspigolon/.npmrc

//registry.npmjs.org/:_authToken = (protected) 
access = null 
if-present = true 
init-license = "MIT" 

; "project" config from /Users/mspigolon/workspace/xxxcutxxxxx/.npmrc

@xxxcutxxxxx:registry = "https://npm.xxxcutxxxxx.net" 
//npm.xxxcutxxxxx.net/:_authToken = (protected) 

; node bin location = /Users/mspigolon/.nvm/versions/node/v17.4.0/bin/node
; cwd = /Users/mspigolon/workspace/xxxcutxxxxx
; HOME = /Users/mspigolon
; Run `npm config ls -l` to show all defaults.

; "publishConfig" from /Users/mspigolon/workspace/xxxcutxxxxx/package.json
; This set of config values will be used at publish-time.

registry = "https://npm.xxxcutxxxxx.net"
@Eomm Eomm added Bug thing that needs fixing Needs Triage needs review for next steps Release 8.x work is associated with a specific npm 8 release labels Jan 24, 2022
@ljharb
Copy link
Contributor

ljharb commented Jan 24, 2022

npm 3.8.1?

@Eomm
Copy link
Contributor Author

Eomm commented Jan 24, 2022

Yes, npm 3.8.1

This is the publish output without the dry-run:

image

with dry-run:

image

@ljharb
Copy link
Contributor

ljharb commented Jan 24, 2022

@Eomm i'm confused; npm 3 is very, very ancient, and node 17 ships with npm 8. Do you mean v8.3.1?

@Eomm
Copy link
Contributor Author

Eomm commented Jan 24, 2022

Oh sorry, I missed the typo.. twice! 😂

Let me copy-paste it:

➜ npm -v       
8.3.1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug thing that needs fixing Release 8.x work is associated with a specific npm 8 release
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants