-
-
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
pages*: use manned.org as much as possible (part 1) #13711
pages*: use manned.org as much as possible (part 1) #13711
Conversation
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. |
@@ -3,7 +3,7 @@ | |||
> Arch Linux package manager utility. | |||
> See also: `pacman-database`, `pacman-deptest`, `pacman-files`, `pacman-key`, `pacman-mirrors`, `pacman-query`, `pacman-remove`, `pacman-sync`, `pacman-upgrade`. | |||
> For equivalent commands in other package managers, see <https://wiki.archlinux.org/title/Pacman/Rosetta>. | |||
> More information: <https://man.archlinux.org/man/pacman.8>. | |||
> More information: <https://manned.org/pacman.8>. |
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.
According to the current style guide, linking to the author's provided man page is preferred.
On the
More information
link line, we prefer linking to the author's provided documentation of the command-line reference or the man page.
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.
Then we should be linking to https://pacman.archlinux.page/pacman.8.html
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.
spanish chapter: LGTM
I just saw a page you might have missed: |
I know there are still pages left. I used this list:
Since the page you point out, is manpages.debian.org, I didn't spot it. The list of changed files is already a lot to be reviewed. We definitely need a new PR to fix the remaining URLs. |
I did
Full list of the resulting lines if you want to check it |
Thank you for providing the list. I want to point out that I focussed on Linux. Looking at man.freebsd.org e.g., I found out that other platforms used those site as well (2 pages in netbsd and 1 in osx). I renamed this PR to part 1. There is still some work to do to reach a uniform way. |
Since this PR is maybe a little too big, I created #13725 to replace manpages.debian.org with manned.org. That PR will contain some pages also included in this PR. Therefore I mark this PR as draft. |
Couple more Although I'm not sure about some of these. Some of these are kinda first party sources but they contain manpages that can also be found in manned Then there are www.unix.com links which return 404 |
This comment has been minimized.
This comment has been minimized.
I ran all "More information" links through |
Thanks, the scope of this PR is to uniform the Linux URLs, so the non-Linux ones should be addressed in another PR. FYI, in #12506 there is a Python-script that I use to check all links |
Yeah, it wasn't intended for this PR. I just thought it would be nice if all these manpage TODOs were collected in the same place. |
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. |
Please see tldr-pages/tldr-maintenance#129, which gets updated each Monday using lychee as well. |
common
,linux
,osx
,windows
,sunos
,android
, etc.