-
-
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
print: add page #15597
print: add page #15597
Conversation
The first time struggle is real. Welcome to the project. |
I had to recover my work from a screenshot I had and it messed up the formatting haha. Thx for the welcome! |
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.
@Managor do you think it will conflict with pages/Linux/print.md
because they have the same name?
It will. Platform specific pages will overshadow common pages. Judging by the content, I'm for removing |
There's also a pages/windows/print.md. I considered submitting it to pages/osx/print.md as zsh is the default MacOS shell, but it's also default on notable Linux distributions. If you stumble across 'print' it's hard to find documentation for it and it's not obvious that it's a zsh builtin. There isn't a dedicated manpage, trying '-h' or '--help' as flags provides no help, and it's generic name makes it more difficult to find information on by Googling or asking an LLM. |
Does EDIT: Also try |
As far as I understand, we have a conflict between run-mailcap's To this end, I suggest:
# print
> Multiple commands have the name `print`.
> See also `print-zsh` and `mailcap-run-print`.
- View tldr page for zsh's print command:
`tldr print-zsh`
- View tldr page for mailcap's print alias:
`tldr run-mailcap-print` |
Will this be a standard in the project for resolving conflicts between other commands that share the same name and platform? |
@gsspdev Would you like to implement this or pass the task to someone else? |
While the last option does cleanly resolve the difference I don't think it's useful to users. The reason for this is as I described before, it is not obvious that 'print' is a zsh command. In the interest of tldr's intent, to provide short descriptions of what something is and how to use it, I think prioritizing zsh's print as I implemented it and moving the current version to 'print-mailcap' makes the most sense. If you use the usual 'info', 'man', and 'help' commands it brings up a better (and already quite short) documentation on how 'print' is an alias of 'mailcap-print'. Tldr provides less utility here. I haven't tried 'info' and 'help' (except as a flag) on zsh's 'print', but could test these. The main concern I have with the proposal is again that a user trying to find information about 'print' is very likely not to know that is a 'zsh' command and would not know which documentation to look in. Another concern is that I think it's important to include the note of 'printf' being the more widely used standard for shell scripting with format strings. The spiritual question to be asked is should tldr prioritize documentation that is comprehensive or simple. I propose adding a short note making the user aware of there also being the mailcap command page which has been moved to a new page and adding that to the see also. I'm willing to create the pull request if others are on board. |
I agree with your sentiment. I have never heard of mailcap before and if someone is looking for mailcap instructions, they probably already know the difference. |
I went and tested help, info and the others. None of them provided with any meaningful insight. |
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.
In terms of this page, this seems to be fine. Lets push this and then make another pull that addresses the overlap. Discuss the issue in #15605
Note that print also exists in the The net effect of merging this PR is that it will not be visible to users on I don't see why we can't merge this now, but just something to keep in mind. |
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.
Just got a single comment.
Co-authored-by: Starbeamrainbowlabs <[email protected]>
common
,linux
,osx
,windows
,sunos
,android
, etc.