Skip to content

Add link checker#21

Merged
adamblanchard merged 1 commit intomy-various-prsfrom
link-checker
Apr 27, 2025
Merged

Add link checker#21
adamblanchard merged 1 commit intomy-various-prsfrom
link-checker

Conversation

@rvedotrc
Copy link
Contributor

@rvedotrc rvedotrc commented Apr 22, 2025

Checks that local markdown links to other markdown docs, or to images, are valid. i.e. the file being linked to actually exists.

See #23 for what the fixes look like.

@rvedotrc rvedotrc marked this pull request as ready for review April 22, 2025 08:49
@rvedotrc rvedotrc requested a review from a team as a code owner April 22, 2025 08:49
@adamblanchard
Copy link
Contributor

This is awesome :D Really cool.

So after reviews, you want to force merge this despite failing Validate links check? And fix them after in #23?

@rvedotrc
Copy link
Contributor Author

This is awesome :D Really cool.

Yay!

So after reviews, you want to force merge this despite failing Validate links check? And fix them after in #23?

Yeah I think that's best. Once it's in place and passing, then it can be set as a required check.

@rvedotrc
Copy link
Contributor Author

you want to force merge this

It wouldn't need a force-merge, I think? AFAICT, neither of the checks (linter, and link checker) have been marked as required checks yet.

@rvedotrc rvedotrc force-pushed the link-checker branch 2 times, most recently from 92d61a3 to a9f945e Compare April 25, 2025 17:15
@rvedotrc rvedotrc changed the base branch from main to my-various-prs April 25, 2025 17:16
@adamblanchard adamblanchard merged commit f415e6e into my-various-prs Apr 27, 2025
1 check passed
@rvedotrc rvedotrc deleted the link-checker branch April 27, 2025 09:45
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

Comments