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.

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

bully: add page #9561

Merged
merged 4 commits into from
Dec 1, 2022
Merged

bully: add page #9561

merged 4 commits into from
Dec 1, 2022

Conversation

TerribleCodes
Copy link
Contributor

  • 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 PR title conforms to the recommended templates.
  • Version of the command being documented :v1.4-00
    Let's document: pen-testing tools #6863

@github-actions github-actions bot added the new command Issues requesting creation of a new page or PRs adding a new page for a command. label Nov 27, 2022
kbdharun
kbdharun reviewed Nov 27, 2022
View reviewed changes
Copy link
Member

@kbdharun kbdharun left a comment

Choose a reason for hiding this comment

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

Thanks for your contribution. I have some suggestions for this page. Also, If possible can you add some more commands or examples to this page?

@kbdharun kbdharun mentioned this pull request Nov 27, 2022
Open
TerribleCodes and others added 2 commits November 27, 2022 22:11
@TerribleCodes @kbdharun
Apply suggestions from code review
493235b 
Co-authored-by: K.B.Dharun Krishna <kbdharunkrishna@gmail.com>
@TerribleCodes
Update bully.md
6deed47
@kbdharun kbdharun requested a review from mfrw November 27, 2022 17:47
EmilyGraceSeville7cf
EmilyGraceSeville7cf approved these changes Nov 28, 2022
View reviewed changes
Copy link
Contributor

@EmilyGraceSeville7cf EmilyGraceSeville7cf left a comment

Choose a reason for hiding this comment

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

  • Approved, no mandatory fixes found.
  • This page should be reviewed by bully users.
  • Thanks for a contribution! 🚀

@TerribleCodes
Update pages/linux/bully.md
6807101 
Co-authored-by: Emily Grace Seville <EmilySeville7cfg@gmail.com>
kbdharun
kbdharun approved these changes Dec 1, 2022
View reviewed changes
Copy link
Member

@kbdharun kbdharun left a comment

Choose a reason for hiding this comment

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

LGTM, Thanks for your contribution.

@EmilyGraceSeville7cf EmilyGraceSeville7cf merged commit 312475c into tldr-pages:main Dec 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@EmilyGraceSeville7cf EmilyGraceSeville7cf EmilyGraceSeville7cf approved these changes

@kbdharun kbdharun kbdharun approved these changes

@mfrw mfrw Awaiting requested review from mfrw

Assignees
No one assigned
Labels
new command Issues requesting creation of a new page or PRs adding a new page for a command.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@TerribleCodes @kbdharun @EmilyGraceSeville7cf