Skip to content

mp3info, zbd, sfill: add mnemonic hints and option placeholders #16919

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 9 commits into
base: main
Choose a base branch
from
Open

mp3info, zbd, sfill: add mnemonic hints and option placeholders #16919

wants to merge 9 commits into from

Conversation

AbhisekLimbu
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 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 page edit Changes to an existing page(s). review needed Prioritized PRs marked for reviews from maintainers. labels Jun 21, 2025
@tldr-bot
Copy link

The build for this PR failed with the following error(s):

pages/common/mp3info.md:7: TLDR014 Page should contain no trailing whitespace
pages/common/mp3info.md:11: TLDR014 Page should contain no trailing whitespace
pages/common/mp3info.md:15: TLDR014 Page should contain no trailing whitespace
pages/common/mp3info.md:19: TLDR014 Page should contain no trailing whitespace
pages/common/mp3info.md:23: TLDR014 Page should contain no trailing whitespace
pages/common/mp3info.md:27: TLDR014 Page should contain no trailing whitespace

Please fix the error(s) and push again.

@AbhisekLimbu
Copy link
Contributor Author

If this format is correct, I’d be happy to continue working on the remaining mnemonic additions starting Monday. I’m looking forward to completing the rest soon. Please let me know if anything needs adjusting!

@Managor
Copy link
Collaborator

Managor commented Jun 22, 2025

That looks very good. I'll link the issue here #16218

@Managor Managor changed the title Add mnemonic hints for short options in mp3info command mp3info: add mnemonic hints Jun 22, 2025
@Managor Managor changed the title mp3info: add mnemonic hints mp3info, zbd, zegrep, sfill: add mnemonic hints Jun 23, 2025
Managor
Managor requested changes Jun 23, 2025
View reviewed changes
Copy link
Collaborator

@Managor Managor left a comment

Choose a reason for hiding this comment

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

Open a new pull request next time for new pages

Managor
Managor reviewed Jun 23, 2025
View reviewed changes
Managor
Managor reviewed Jun 23, 2025
View reviewed changes
@Managor Managor changed the title mp3info, zbd, zegrep, sfill: add mnemonic hints mp3info, zbd, sfill: add mnemonic hints Jun 23, 2025
@Managor Managor changed the title mp3info, zbd, sfill: add mnemonic hints mp3info, zbd, sfill: add mnemonic hints and option placeholders Jun 23, 2025
Managor
Managor approved these changes Jun 23, 2025
View reviewed changes
Copy link
Collaborator

@Managor Managor left a comment

Choose a reason for hiding this comment

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

Looks good to go.

@AbhisekLimbu
Copy link
Contributor Author

Thank you!!!

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

@Managor Managor Managor approved these changes

At least 2 approving reviews are required to merge this pull request.

Assignees
No one assigned
Labels
page edit Changes to an existing page(s). 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.
3 participants
@AbhisekLimbu @tldr-bot @Managor