Skip to content

npm-ping: add page #19055

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 12 commits into
base: main
Choose a base branch
from
Open

npm-ping: add page #19055

wants to merge 12 commits into from
+12 −1

Conversation

@ishaa05
Copy link

@ishaa05 ishaa05 commented Oct 27, 2025

If you are a new contributor to the project, do not use AI to generate pages. We will close any PR with a suspicion of AI usage.

  • The page(s) are in the correct platform directories: common, linux, osx, windows, sunos, android, etc.
  • The page description(s) have links to documentation or a homepage.
  • The page(s) follow the content guidelines.
  • The page(s) follow the style guide.
  • The PR contains at most 5 new pages.
  • The PR title conforms to the recommended templates.
  • Version of the command being documented (if known):

@github-actions github-actions bot added new command Issues requesting creation of a new page or PRs adding a new page for a command. review needed Prioritized PRs marked for reviews from maintainers. labels Oct 27, 2025
sebastiaanspeck
sebastiaanspeck requested changes Oct 27, 2025
View reviewed changes
@sebastiaanspeck sebastiaanspeck changed the title Add npm-ping: document npm registry ping command npm-ping: add page Oct 27, 2025
@sebastiaanspeck sebastiaanspeck mentioned this pull request Oct 27, 2025
Open
67 tasks
sebastiaanspeck
sebastiaanspeck requested changes Oct 27, 2025
View reviewed changes
@ishaa05 @sebastiaanspeck
Update pages/common/npm-ping.md
f87ad15 
Co-authored-by: Sebastiaan Speck <12570668+sebastiaanspeck@users.noreply.github.com>
dmmqz
dmmqz reviewed Oct 27, 2025
View reviewed changes
pages.en Outdated
Copy link
Member

Choose a reason for hiding this comment

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

Remove this change from the PR.

ishaa05 and others added 4 commits October 28, 2025 00:37
@ishaa05 @dmmqz
Update pages/common/npm-ping.md
0c216e3 
Co-authored-by: Dylan <145150333+dmmqz@users.noreply.github.com>
@ishaa05 @dmmqz
Update pages/common/npm-ping.md
ee56e97 
Co-authored-by: Dylan <145150333+dmmqz@users.noreply.github.com>
@ishaa05 @dmmqz
Update pages/common/npm-ping.md
4bf095c 
Co-authored-by: Dylan <145150333+dmmqz@users.noreply.github.com>
@ishaa05 @dmmqz
Update pages/common/npm-ping.md
9022beb 
Co-authored-by: Dylan <145150333+dmmqz@users.noreply.github.com>
sebastiaanspeck
sebastiaanspeck requested changes Oct 27, 2025
View reviewed changes
pages.en Outdated
Copy link
Member

Choose a reason for hiding this comment

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

Please revert this deletion.

@acuteenvy
Copy link
Member

This is a duplicate of #18974.

sebastiaanspeck
sebastiaanspeck approved these changes Oct 28, 2025
View reviewed changes
@sebastiaanspeck
Copy link
Member

This is a duplicate of #18974.

True, but that one hasn't signed the CLA yet.

ishaa05 and others added 2 commits October 28, 2025 13:15
@ishaa05 @sebastiaanspeck
Update pages/common/npm-ping.md
f7d6528 
Co-authored-by: Sebastiaan Speck <12570668+sebastiaanspeck@users.noreply.github.com>
@ishaa05
Merge branch 'main' into npmping
49be1fe
Managor
Managor requested changes Oct 28, 2025
View reviewed changes
pages.en
Copy link
Member

Choose a reason for hiding this comment

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

Undo this deletion

sebastiaanspeck
sebastiaanspeck requested changes Oct 28, 2025
View reviewed changes
Copy link
Member

@sebastiaanspeck sebastiaanspeck left a comment

Choose a reason for hiding this comment

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

Undo the deletion of pages.en.

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

Reviewers

@dmmqz dmmqz dmmqz left review comments

@sebastiaanspeck sebastiaanspeck sebastiaanspeck requested changes

@Managor Managor Managor requested changes

Requested changes must be addressed to merge this pull request.

Assignees

No one assigned

Labels

new command Issues requesting creation of a new page or PRs adding a new page for a command. review needed Prioritized PRs marked for reviews from maintainers.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@ishaa05 @acuteenvy @sebastiaanspeck @Managor @dmmqz