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

Script to check links #111

Closed
wants to merge 6 commits into from
Closed

Script to check links #111

wants to merge 6 commits into from

Conversation

briandfoy
Copy link

A script to check some links, and a lot of link fixes.

Many of the links upgraded to https. I removed some stuff that disappeared.

This should fix #7, #64, and #84.

bin/check_links Show resolved Hide resolved
@karenetheridge
Copy link
Member

Test::Pod::No404s exists - and can be used via its dzil plugin.

@briandfoy
Copy link
Author

Many of the links that were updated did not return 404. They were mostly 302 or 500. Either way, if you want to do it another way that's fine, but here's the fix for several tickets even if you don't want my script.

@briandfoy
Copy link
Author

What needs to happen for this to move forward?

@haarg
Copy link

haarg commented Sep 6, 2023

The bin/check_links script needs to either be removed from the PR or excluded from the built dist.

It can be excluded from the dist by adding a line exclude_filename = bin/check_links to the [Git::GatherDir] section in dist.ini.

@briandfoy
Copy link
Author

I've excluded the script in dist.ini. What else needs to happen for this to move forward?

@briandfoy briandfoy closed this by deleting the head repository Oct 15, 2023
@wchristian
Copy link
Member

Thanks @karenetheridge

Test::Pod::No404s exists - and can be used via its dzil plugin.

I'll be setting that up since, as astute readers of its docs may know, it actually handles more than just 404.

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.

Link check tests
6 participants