Skip to content

Conversation

savvyspoon
Copy link
Contributor

@savvyspoon savvyspoon commented Sep 24, 2020

  • The page (if new), does not already exist in the repo.
  • The page is in the correct platform directory (common/, linux/, etc.)
  • The page has 8 or fewer examples.
  • The PR title conforms to the recommended templates.
  • The page follows the content guidelines.
  • The page description includes a link to documentation or a homepage (if applicable).

Fixes #4115

@CLAassistant
Copy link

CLAassistant commented Sep 24, 2020

CLA assistant check
All committers have signed the CLA.

@tldr-bot
Copy link

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

pages/linux/semanage.md:25: TLDR008 File should contain no trailing whitespace

Please fix the error(s) and push again.

@navarroaxel navarroaxel added the new command Issues requesting creation of a new page or PRs adding a new page for a command. label Sep 24, 2020
Copy link
Member

@sbrl sbrl 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 the new page, @savvyspoon! I've left some comments below for you to review.


`semanage -S {{store}} -o {{output_file}}`

- Input local customizations:
Copy link
Member

Choose a reason for hiding this comment

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

What precisely do you mean by this?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Take a set of commands from a specified file and load them in a single transaction. May be a better way of putting it

Copy link
Member

Choose a reason for hiding this comment

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

Ah, I see. Thanks for the clarification!

Import would work too, but your phrase is fine :-)

savvyspoon and others added 7 commits September 29, 2020 15:12
Co-authored-by: Starbeamrainbowlabs <[email protected]>
Co-authored-by: Starbeamrainbowlabs <[email protected]>
Co-authored-by: Starbeamrainbowlabs <[email protected]>
Co-authored-by: Starbeamrainbowlabs <[email protected]>
Co-authored-by: Starbeamrainbowlabs <[email protected]>
Co-authored-by: Starbeamrainbowlabs <[email protected]>
@navarroaxel navarroaxel requested a review from sbrl September 30, 2020 11:36
Copy link
Member

@sbrl sbrl left a comment

Choose a reason for hiding this comment

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

Looks ok to me! Thanks, @savvyspoon

@github-actions
Copy link

Hi all! This thread has not had any recent activity.
Are there any updates? Thanks!

@github-actions github-actions bot added the waiting Issues/PRs with Pending response by the author. label Oct 17, 2020
@navarroaxel navarroaxel merged commit 10a79e1 into tldr-pages:master Oct 17, 2020
@bl-ue bl-ue removed the waiting Issues/PRs with Pending response by the author. label Mar 3, 2021
@savvyspoon savvyspoon deleted the patch-1 branch July 2, 2023 22:40
@Managor Managor mentioned this pull request Oct 7, 2024
43 tasks
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.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

page request: semanage

6 participants