Skip to content

Fix broken examples in npm registry docs - #45875

Merged
mchammer01 merged 1 commit into
github:mainfrom
Joshwanda17:fix-npm-registry-docs
Sep 14, 2026
Merged

mchammer01 merged 1 commit into
github:mainfrom
Joshwanda17:fix-npm-registry-docs

Conversation

@Joshwanda17

Copy link
Copy Markdown
Contributor

Fixes three issues in working-with-the-npm-registry.md:

  • 'Installing packages from other organizations' showed the identical @namespace line twice instead of two distinct examples, defeating the purpose of the section
  • Stray space in the GHES publishConfig registry URL (npm. HOSTNAME/)
  • Double slash in the GHES .npmrc authToken line, which would prevent npm from matching the token to the registry

@github-actions github-actions Bot added the triage Do not begin working on this issue until triaged by the team label Sep 14, 2026
@github-actions

Copy link
Copy Markdown
Contributor

How to review these changes 👓

Thank you for your contribution. To review these changes, choose one of the following options:

A Hubber will need to deploy your changes internally to review.

Table of review links

Note: Please update the URL for your staging server or codespace.

The table shows the files in the content directory that were changed in this pull request. This helps you review your changes on a staging server. Changes to the data directory are not included in this table.

Source Review Production What Changed
packages/working-with-a-github-packages-registry/working-with-the-npm-registry.md fpt
ghec
ghes@ 3.22 3.21 3.20 3.19 3.18 3.17
fpt
ghec
ghes@ 3.22 3.21 3.20 3.19 3.18 3.17

Key: fpt: Free, Pro, Team; ghec: GitHub Enterprise Cloud; ghes: GitHub Enterprise Server

🤖 This comment is automatically generated.

@armanayub1973

Copy link
Copy Markdown

Please sir I need you help my country Bangladesh

@janiceilene

Copy link
Copy Markdown
Contributor

Thanks for opening a PR @Joshwanda17! I'll get this triaged to the team for review.

@mchammer01 mchammer01 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Joshwanda17 👋🏻 - thank you for these fixes, LGTM ✨
Let's merge this :shipit:

@mchammer01
mchammer01 added this pull request to the merge queue Sep 14, 2026
Merged via the queue into github:main with commit c07cd52 Sep 14, 2026
59 checks passed
@github-actions

Copy link
Copy Markdown
Contributor

Thanks very much for contributing! Your pull request has been merged 🎉 You should see your changes appear on the site in approximately 24 hours. If you're looking for your next contribution, check out our help wanted issues

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

builder persona triage Do not begin working on this issue until triaged by the team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants