Skip to content
This repository has been archived by the owner on Feb 5, 2025. It is now read-only.

Check all links #15

Merged
merged 56 commits into from
Jan 7, 2025
Merged

Check all links #15

merged 56 commits into from
Jan 7, 2025

Conversation

ealmloff
Copy link
Member

@ealmloff ealmloff commented Dec 31, 2024

Stacked on #14 which is currently used in the docsite. This PR adds checking for all relative links by expanding links to the Route enum instead of string links

Closes #15

TODO:

  • Also check all fragments

@ealmloff ealmloff marked this pull request as ready for review January 2, 2025 18:44
@ealmloff ealmloff added the enhancement New feature or request label Jan 2, 2025
@Andrew15-5
Copy link

You've added a missing space instead of soft break in 80a1aad, but it didn't affect the test from 2620d7b?

@ealmloff
Copy link
Member Author

ealmloff commented Jan 7, 2025

You've added a missing space instead of soft break in 80a1aad, but it didn't affect the test from 2620d7b?

Yes, that test is fixed now

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants