Skip to content

coreutils*: use short more info links #5658

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 1 commit into from
Apr 1, 2021

Conversation

vladimyr
Copy link
Contributor

@vladimyr vladimyr commented Apr 1, 2021

  • The page (if new), does not already exist in the repo.
  • The page is in the correct platform directory (common/, linux/, etc.)
  • The page has 8 or fewer examples.
  • The PR title conforms to the recommended templates.
  • The page follows the content guidelines.
  • The page description includes a link to documentation or a homepage (if applicable).

GNU online documentation provides user-friendly URLs - for example, https://www.gnu.org/software/coreutils/ls which redirects to https://www.gnu.org/software/coreutils/manual/html_node/ls-invocation.html#ls-invocation. Those short forms are advertised through coreutils manpages (for example, ls) so I've updated all accompanying coreutils tldr pages to follow the suit.

This is done using following script:

rg -l 'gnu.org/software/coreutils/manual/html_node/(.*?)-invocation.html' |
    xargs sed -i -E 's|(gnu.org/software/coreutils)/manual/html_node/(.*?)-invocation.html|\1/\2|g'

Copy link
Member

@tansiret tansiret left a comment

Choose a reason for hiding this comment

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

LGTM.

@bl-ue bl-ue added mass changes Changes that affect multiple pages. page edit Changes to an existing page(s). translation Issues requesting translating pages from English to other languages. labels Apr 1, 2021
Copy link
Contributor

@bl-ue bl-ue left a comment

Choose a reason for hiding this comment

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

Wow, really cool!

@vladimyr
Copy link
Contributor Author

vladimyr commented Apr 1, 2021

Thanks, I'd do anything to have shorter URLs not spanning multiple lines in my terminal 😀

@tansiret tansiret merged commit 4830093 into tldr-pages:master Apr 1, 2021
@vladimyr vladimyr deleted the coreutils branch April 1, 2021 16:04
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
mass changes Changes that affect multiple pages. page edit Changes to an existing page(s). translation Issues requesting translating pages from English to other languages.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants