Skip to content

aa-*, apparmor_parser: add page #16844

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
Merged
merged 14 commits into from
Jun 14, 2025
Merged

aa-*, apparmor_parser: add page #16844

merged 14 commits into from
Jun 14, 2025

Conversation

Prithviraj03
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): Unknown

Added TLDR pages for the following AppArmor commands as part of issue #3689:

  • aa-audit
  • aa-cleanprof
  • aa-decode
  • aa-genprof
  • aa-logprof
  • aa-mergeprof
  • aa-remove-unknown
  • aa-teardown
  • aa-unconfined
  • apparmor_parser

@github-actions github-actions bot added mass changes Changes that affect multiple pages. new command Issues requesting creation of a new page or PRs adding a new page for a command. page edit Changes to an existing page(s). review needed Prioritized PRs marked for reviews from maintainers. labels Jun 13, 2025
Managor
Managor requested changes Jun 13, 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.

Prefer using longform options or use option placeholders.

@Managor Managor changed the title Add apparmor commands aa-*, apparmor_parser: add page Jun 13, 2025
sebastiaanspeck
sebastiaanspeck requested changes Jun 13, 2025
View reviewed changes
@Prithviraj03
docs: update aa-audit, aa-cleanprof, aa-decode, aa-genprof, aa-logpro…
7386099 
…f, aa-mergeprof, aa-remove-unknown, aa-teardown, aa-unconfined, apparmor_parser.md command syntax and options
sebastiaanspeck
sebastiaanspeck requested changes Jun 13, 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.

Replace all Display help information with Display help

@Prithviraj03
Copy link
Contributor Author

@sebastiaanspeck Thanks for reminding me again I made changes according to the rules.

Managor
Managor approved these changes Jun 13, 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 after this

@Prithviraj03 @Managor
fix: minor formatting error
9bed426 
Co-authored-by: Managor <42655600+Managor@users.noreply.github.com>
spageektti
spageektti reviewed Jun 13, 2025
View reviewed changes
@spageektti
Copy link
Member

Hi. Thank you so much for your contribution and welcome to tldr!

spageektti and others added 6 commits June 13, 2025 20:46
@spageektti
Merge branch 'main' into add-apparmor-commands
e33cd91
@Prithviraj03 @spageektti
fix: format
360ee2f 
Co-authored-by: Wiktor Perskawiec <wiktor@perskawiec.cc>
@Prithviraj03 @spageektti
fix: format
e0b1c19 
Co-authored-by: Wiktor Perskawiec <wiktor@perskawiec.cc>
@Prithviraj03 @spageektti
fix: format
72916ed 
Co-authored-by: Wiktor Perskawiec <wiktor@perskawiec.cc>
@Prithviraj03 @spageektti
fix: format
8ac0b51 
Co-authored-by: Wiktor Perskawiec <wiktor@perskawiec.cc>
@Prithviraj03 @spageektti
fix: format
911ce16 
Co-authored-by: Wiktor Perskawiec <wiktor@perskawiec.cc>
@Managor Managor merged commit 529b37f into tldr-pages:main Jun 14, 2025
4 checks passed
This was referenced Jun 18, 2025
Closed
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@spageektti spageektti spageektti left review comments

@sebastiaanspeck sebastiaanspeck sebastiaanspeck approved these changes

@Managor Managor Managor approved these changes

Assignees
No one assigned
Labels
mass changes Changes that affect multiple pages. new command Issues requesting creation of a new page or PRs adding a new page for a command. 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.
4 participants
@Prithviraj03 @spageektti @sebastiaanspeck @Managor