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

ffuf: add page #12082

Merged
merged 7 commits into from
Jan 23, 2024
Merged

ffuf: add page #12082

merged 7 commits into from
Jan 23, 2024

Conversation

p3rception
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 (if known):

@github-actions github-actions bot added the new command Issues requesting creation of a new page. label Jan 17, 2024
@tldr-bot

This comment was marked as outdated.

Sign in to view
@tldr-bot

This comment was marked as outdated.

Sign in to view
sebastiaanspeck
sebastiaanspeck reviewed Jan 18, 2024
View reviewed changes
pages/common/ffuf.md Outdated Show resolved Hide resolved
pages/common/ffuf.md Outdated Show resolved Hide resolved
gutjuri
gutjuri reviewed Jan 18, 2024
View reviewed changes
Copy link
Member

@gutjuri gutjuri left a comment

Choose a reason for hiding this comment

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

Thanks, I have some minor suggestions.

pages/common/ffuf.md Outdated Show resolved Hide resolved
pages/common/ffuf.md Outdated Show resolved Hide resolved
kbdharun
kbdharun approved these changes Jan 18, 2024
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.

pages/common/ffuf.md Outdated Show resolved Hide resolved
gutjuri
gutjuri approved these changes Jan 18, 2024
View reviewed changes
Copy link
Member

@gutjuri gutjuri left a comment

Choose a reason for hiding this comment

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

Good!

acuteenvy
acuteenvy reviewed Jan 18, 2024
View reviewed changes
pages/common/ffuf.md Outdated Show resolved Hide resolved
pages/common/ffuf.md Outdated Show resolved Hide resolved
vitorhcl
vitorhcl approved these changes Jan 18, 2024
View reviewed changes
Copy link
Member

@vitorhcl vitorhcl 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 suggested some more mnemonics.

pages/common/ffuf.md Outdated Show resolved Hide resolved
pages/common/ffuf.md Outdated Show resolved Hide resolved
pages/common/ffuf.md Outdated Show resolved Hide resolved
vitorhcl
vitorhcl reviewed Jan 18, 2024
View reviewed changes
pages/common/ffuf.md Outdated Show resolved Hide resolved
@p3rception
Copy link
Contributor Author

Needs anything else for merging?

@acuteenvy acuteenvy merged commit 68e7eee into tldr-pages:main Jan 23, 2024
4 checks passed
@gutjuri gutjuri mentioned this pull request Apr 14, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@vitorhcl vitorhcl vitorhcl approved these changes

@kbdharun kbdharun kbdharun approved these changes

@sebastiaanspeck sebastiaanspeck sebastiaanspeck left review comments

@acuteenvy acuteenvy acuteenvy approved these changes

@gutjuri gutjuri gutjuri approved these changes

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

Successfully merging this pull request may close these issues.
7 participants
@p3rception @tldr-bot @sebastiaanspeck @gutjuri @kbdharun @vitorhcl @acuteenvy