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

Replace many dead links with web.archive.org equivalants. #397

Merged
merged 4 commits into from
May 13, 2023

Conversation

IoIxD
Copy link
Contributor

@IoIxD IoIxD commented May 10, 2023

There was quite a few dead links, especially at the beginning. This PR replaces them with the most recent web.archive.org snapshot

@catwell
Copy link
Member

catwell commented May 10, 2023

It would probably be better to replace them with the new links for those projects when they exist, especially when they are lunarmodules projects!

I.e.:

The textui docs link (https://nilnor.github.io/textui/docs/) appears to work for me.

Using archive.org for z/OS might make sense, or we may just remove the reference.

Copy link
Member

@alerque alerque left a comment

Choose a reason for hiding this comment

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

Much like @catwell suggested it would be much better to link to the new homes of these projects rather than to archived versions of their old homes. Even if the exact same page is not available we should adjust the references so that we link to current resources. If anything is really no longer available we should probably just rewrite the docs to avoid the reference and substitute something current or at least make sure it is a minor point.

Thanks for taking the time to contribute. Are you able to update this PR with current links?

@IoIxD
Copy link
Contributor Author

IoIxD commented May 11, 2023

Swear it didn't work for me, but yeah I guess it does. Fixed.

manual.md Outdated Show resolved Hide resolved
Co-authored-by: Caleb Maclennan <caleb@alerque.com>
Copy link
Contributor Author

@IoIxD IoIxD left a comment

Choose a reason for hiding this comment

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

Done.

Copy link
Member

@alerque alerque left a comment

Choose a reason for hiding this comment

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

Thanks for the contribution.

@alerque alerque merged commit e4940da into lunarmodules:master May 13, 2023
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants