Skip to content
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

docs: Normalize links to https or their web archived version when unavailable #1236

Merged
merged 8 commits into from
Nov 11, 2024

Conversation

echoix
Copy link
Member

@echoix echoix commented Nov 11, 2024

I had worked on last February to change http links to https when possible. Down that rabbit hole, I opened every link I saw to see if it had a https version, and when the link was dead, I made a separate research to find where that content is found now. When it was impossible to find an equivalent content hosted somewhere, I changed the link to a link to the wayback machine. Since I used the Internet Archive extension at that time, I also made sure that every link I opened was at least backed up in the last 90 days, as of February 2024. I also used the wayback machine to help find the location of a moved website that didn't have redirects anymore.

I was extensively using search and replace, and also normalized multiple different URLs from different files to be uniformized across the repo, when it made sense. Example: taking permalinks when available, using the recommended URL scheme for publication ids, DOI, etc.

I had an open PR in my fork of this work where I asked for some feedback before posting here.

I'm opening the PR right away, but I'm currently rebasing the commits to remove already committed fixes (to enable linting), and some URLs recently independently updated.

@echoix echoix added the manual Documentation related issues label Nov 11, 2024
@echoix echoix requested a review from neteler November 11, 2024 03:09
@echoix echoix force-pushed the https-links branch 2 times, most recently from 82f490a to 5b24c47 Compare November 11, 2024 04:15
Copy link
Member

@neteler neteler left a comment

Choose a reason for hiding this comment

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

With the few suggested modifications applied good to go.

@echoix echoix merged commit 59e289f into OSGeo:grass8 Nov 11, 2024
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
manual Documentation related issues
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants