-
-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
pacman-s: add alias page; pacman-sync: update page #14658
Conversation
@Managor and @cyqsimon |
I'm a bit torn on this as well. We currently don't seem to have a convention for commands whose primary verb is a flag, as is the case with |
Why not rename it to pacman-s.md? Or pacman-S.md? |
@sebastiaanspeck @cyqsimon |
No, no need for that. I was just pointing out that we probably should standardise this sort of thing. |
The way I see it is that the best way to handle this is to edit the commands in pacman-sync.md to match what people use. Leave the filename and title of that page as is. pacman.md already has a suggestion to check out the pacman-sync page. Edit this page to be an alias of that page. |
Just to clarify, |
What do you mean with "full form commands in the pacman-s page"? |
By full form commands, I mean the |
No, just make this page into an alias. See other pages like https://github.com/tldr-pages/tldr/blob/main/pages/common/trash-cli.md for example. If you want, you can use mnemonics to clarify |
Hello! I've noticed something unusual when checking this PR:
Is this intended? If so, just ignore this comment. Otherwise, please double-check the commits. |
The build for this PR failed with the following error(s):
Please fix the error(s) and push again. |
Co-authored-by: Sebastiaan Speck <12570668+sebastiaanspeck@users.noreply.github.com>
Now all that is left is converting the commands to shortform |
Co-authored-by: Sebastiaan Speck <12570668+sebastiaanspeck@users.noreply.github.com>
Co-authored-by: Sebastiaan Speck <12570668+sebastiaanspeck@users.noreply.github.com>
Co-authored-by: Sebastiaan Speck <12570668+sebastiaanspeck@users.noreply.github.com>
Co-authored-by: Managor <42655600+Managor@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me once the open issues have been resolved. Thanks for taking your time to go through with this.
Remember that none of this is set in stone. If you feel like the format of this page could be improved for clarity, just fire another PR.
Co-authored-by: Managor <42655600+Managor@users.noreply.github.com>
Co-authored-by: Sebastiaan Speck <12570668+sebastiaanspeck@users.noreply.github.com>
Co-authored-by: Managor <42655600+Managor@users.noreply.github.com>
Thank you for the corrections and feedback for the PR. |
There was a problem hiding this 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.
#14003
common
,linux
,osx
,windows
,sunos
,android
, etc.