tldr icon indicating copy to clipboard operation
tldr copied to clipboard

pacman: use short options only

Open marchersimon opened this issue 3 years ago • 5 comments

For #8046 I felt like the whole issue is mostly about pacman. So we should start with this one and see if there still are other commands where it is necessary to include short options. I put the short options in token syntax for now, but we can debate about that.

marchersimon avatar Aug 03 '22 07:08 marchersimon

Personally don't like this current version.

I think we just pick one, and since its such a common command, maybe we should go with sudo pacman -Syu. I've probably said something else in the past as well.

CleanMachine1 avatar Aug 03 '22 23:08 CleanMachine1

I just think that it's useful to have the long version, since you directly know what all options do and what would happen if you left out a specific one. But I totally get your point, that it looks weird.

marchersimon avatar Aug 04 '22 08:08 marchersimon

I think we just pick one, and since it's such a common command, maybe we should go with sudo pacman -Syu. I've probably said something else in the past as well.

Yeah, I think the same too, Even though long options are useful and very informative in the majority of cases. I think many would prefer shorter options for pacman as it is the one that is more widely used than the longer version.

I think the current implementation might confuse users about which command to execute. I went through the issue about it fully and I noticed there were a lot of suggestions and proposed solutions.

I agree with @CleanMachine1 as this is a special case it is better to use shorter options on the page if users need to refer to longer options they can use the man link provided with More information:.

kbdharun avatar Aug 06 '22 15:08 kbdharun

I don't see why we should put this into common, it's only available on linux. Also, if you are on linux and look up the tldr page for pacman, you will only see the linux version, not the common version.

marchersimon avatar Aug 08 '22 12:08 marchersimon

I don't see why we should put this into common, it's only available on linux. Also, if you are on linux and look up the tldr page for pacman, you will only see the linux version, not the common version.

+1, I too think the same.

kbdharun avatar Aug 08 '22 12:08 kbdharun

The pacman page should not be in common. common is only for pages about commands that are supported by more than 1 platform.

@sbrl, what about not to use common at all? It's not clear what platforms on some command is available when it is in common/.

Further, as I have explained in another issue recently, we tldr pages should document usages of commands that are useful. Sure, it's helpful to know what the short options stand for, but there are other ways to achieve that than requiring long options everywhere. sudo pacman --sync --refresh --sysupgrade is an enormous mouthful to remember and type every time - I'll bet most users use sudo pacman -Syu instead.

Let's define subset of pages where we prefer short options. For instance in package manager pages we can prefer short options anywhere. I can't decide whether to approve or decline such PRs without formal rules. I have nothing to refer to. Note that when I block merging by requesting changes it doesn't mean that I don't like changes made in PR at all. I just wanna everything to be standardized and that's why I am here.

EmilyGraceSeville7cf avatar Aug 10 '22 11:08 EmilyGraceSeville7cf

Let's define subset of pages where we prefer short options. For instance in package manager pages we can prefer short options anywhere. I can't decide whether to approve or decline such PRs without formal rules. I have nothing to refer to. Note that when I block merging by requesting changes it doesn't mean that I don't like changes made in PR at all. I just wanna everything to be standardized and that's why I am here.

Yeah, it would be better to specify the use cases for short commands over longer ones for PR's, maybe we can document the edge cases too in contributing.md. What is your opinion? @marchersimon @CleanMachine1 @navarroaxel @sbrl

kbdharun avatar Aug 10 '22 12:08 kbdharun

Yeah, it would be better to specify the use cases for short commands over longer ones for PR's, maybe we can document the edge cases too in contributing.md. What is your opinion? @marchersimon @CleanMachine1

Style guide maybe the better place to document it.

EmilyGraceSeville7cf avatar Aug 10 '22 12:08 EmilyGraceSeville7cf

Style guide maybe the better place to document it.

Yeah, That would be good. To place it under special cases under style guide.

kbdharun avatar Aug 10 '22 12:08 kbdharun