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.

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

chcon: move help strings to a markdown file #4445

Closed
wants to merge 3 commits into from

Conversation

gadcam
Copy link

@gadcam gadcam commented Feb 27, 2023

Moves the about and summary help strings to chcon.md in the chcon directory.

See #4368 for more context.

```
chcon [OPTION]... CONTEXT FILE...
chcon [OPTION]... [-u USER] [-r ROLE] [-l RANGE] [-t TYPE] FILE...
chcon [OPTION]... --reference=RFILE FILE...
Copy link
Contributor

Choose a reason for hiding this comment

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

the spell check is complaining here

Copy link
Author

Choose a reason for hiding this comment

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

Added an ignore at the beginning of the file 85567d5
However I am asking myself whether RFILE should not be added to this list as it's not the only place where it's used. https://github.com/uutils/coreutils/blob/f5a9ffe52fb40c045684ea95129f6020e04bfff2/.vscode/cspell.dictionaries/jargon.wordlist.txt
What do you think ?

Copy link
Member

Choose a reason for hiding this comment

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

Yeah that sounds good!

@github-actions
Copy link

GNU testsuite comparison:

GNU test failed: tests/misc/timeout. tests/misc/timeout is passing on 'main'. Maybe you have to rebase?

@zleyyij
Copy link
Contributor

zleyyij commented Mar 2, 2023

This is addressed by #4437

@tertsdiepraam
Copy link
Member

Indeed those PRs we're doing the same thing. Feel free to pick up other utils (which should go a lot quicker now that we've had a review round here!)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants