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

Complete and automated scan of all 405 urls in the doc folder. Fixed all 404 errors and dead links, removed one or two, a few open issues #81

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

Anynomouss
Copy link

@Anynomouss Anynomouss commented Apr 5, 2024

Open issues, links that were not fixed or removed:

…few miner other fixes. A link problems are unresolved, I will put them in a comment.
@Anynomouss Anynomouss changed the title I made a complete and l automated scan of all 405 doc links. Fixed all 404 errors and dead links, removed one or two, a few open issues I made a complete and automated scan of all 405 doc links. Fixed all 404 errors and dead links, removed one or two, a few open issues Apr 5, 2024
@Anynomouss
Copy link
Author

I made the doc dead link scanner available here. It is far from perfect, but still useful in saving time for a rather boring task such as dead link checking.
https://github.com/Anynomouss/dead_link_checker

@Anynomouss Anynomouss changed the title I made a complete and automated scan of all 405 doc links. Fixed all 404 errors and dead links, removed one or two, a few open issues Complete and automated scan of all 405 urls in the doc folder. Fixed all 404 errors and dead links, removed one or two, a few open issues Apr 5, 2024
Copy link
Author

Choose a reason for hiding this comment

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

Removal of gaeapool.com was an accident, so can be ignored. However, the site is giving an SSL certificate expired error upon checking.

Copy link
Contributor

Choose a reason for hiding this comment

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

https://gaeapool.com/#/
it works for me.

Choose a reason for hiding this comment

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

This new link has some odd characters in it and has text indicating .exe but a link that says .rar (the original also linked to a .rar so that part is consistent)

Is there a better source for this file?

Copy link
Contributor

Choose a reason for hiding this comment

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

No, only official ipollo site has it.

@CircusDad
Copy link

I made the one comment above but otherwise suggest this be merged. This is pretty old. Any reason it is still outstanding?

@Anynomouss
Copy link
Author

Not sure myself why the may pull requests got cued. Is it lack of OC manpower and the need to have at least multiple review from repository owners? If that is the case, I would propose appointing me and others who are willing as additional reviewers. That can be done without changing repository owners. Those left with admin powers can do a final check and do the merging. I think that would be an improvement from the current situation where any proposed changes are ignored for months if not years.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants