Skip to content

ping.py, ping6.py, impacket-{ping, ping6}: add page #16177

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

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

MachiavelliII
Copy link
Member

  • The page(s) are in the correct platform directories: common, linux, osx, windows, sunos, android, etc.
  • The page(s) have at most 8 examples.
  • 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 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 Apr 15, 2025
@MachiavelliII MachiavelliII mentioned this pull request Apr 9, 2025
62 tasks
MachiavelliII and others added 2 commits April 15, 2025 21:45
Co-authored-by: Wiktor Perskawiec <[email protected]>
Co-authored-by: Wiktor Perskawiec <[email protected]>
@tldr-bot

This comment has been minimized.

Comment on lines +4 to +5
> Part of the Impacket suite.
> More information: <https://github.com/fortra/impacket>.
Copy link
Member

Choose a reason for hiding this comment

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

"Part of the Impacket suite" doesn't add much to the description when impacket is already in the command.

More information links have been removed from alias pages (#14542), so unless the alias differs from the original command in a way that would warrant adding additional documentation, this is redundant.

Suggested change
> Part of the Impacket suite.
> More information: <https://github.com/fortra/impacket>.

Comment on lines +4 to +5
> Part of the Impacket suite.
> More information: <https://github.com/fortra/impacket>.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
> Part of the Impacket suite.
> More information: <https://github.com/fortra/impacket>.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
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
Development

Successfully merging this pull request may close these issues.

4 participants