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

Added GitHub action to check links #51

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

joachimmetz
Copy link
Collaborator

@joachimmetz joachimmetz commented Dec 18, 2022

For reviewers don't merge, currently testing GitHub actions

@joachimmetz joachimmetz self-assigned this Dec 18, 2022
@joachimmetz joachimmetz force-pushed the citests1 branch 11 times, most recently from 9c6e301 to 2d1d5b1 Compare December 27, 2022 14:29
@joachimmetz joachimmetz force-pushed the citests1 branch 11 times, most recently from 8d6cafe to 1e4ddf9 Compare January 1, 2023 11:26
@joachimmetz
Copy link
Collaborator Author

joachimmetz commented Jan 1, 2023

The current lint checker seems to have issues with certain URLs and returns a 0, 403, 50x

@joachimmetz joachimmetz force-pushed the citests1 branch 4 times, most recently from f144b35 to 9ecb4a2 Compare January 2, 2023 21:54
@joachimmetz
Copy link
Collaborator Author

joachimmetz commented Jan 3, 2023

Per chat with @rjcolonna mkdocs possibly has a link check feature as well

per comment on https://stackoverflow.com/questions/68577217/mkdocs-check-link-validity this might only be for "internal" links

@joachimmetz
Copy link
Collaborator Author

Looks like myst has some interesting additional linting features

/home/runner/work/wiki/wiki/docs/$mft.md:13: WARNING: Document headings start at H2, not H1 [myst.header]
/home/runner/work/wiki/wiki/docs/$mft.md:36: WARNING: Non-consecutive header level increase; H2 to H4 [myst.header]

@joachimmetz joachimmetz force-pushed the citests1 branch 3 times, most recently from 4cdec53 to 3138716 Compare January 19, 2023 08:47
@joachimmetz joachimmetz force-pushed the citests1 branch 2 times, most recently from 502c550 to 8c2d7a1 Compare January 26, 2023 05:07
@joachimmetz joachimmetz force-pushed the citests1 branch 3 times, most recently from 8a3e92d to 7d2a6ab Compare October 4, 2023 04:22
@joachimmetz joachimmetz force-pushed the citests1 branch 7 times, most recently from ed246cf to b0e4d59 Compare December 27, 2023 09:09
@joachimmetz joachimmetz force-pushed the citests1 branch 3 times, most recently from fb8185f to c77a6c3 Compare January 6, 2024 08:07
@joachimmetz
Copy link
Collaborator Author

Down to 54, some links cannot be correctly evaluated.

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.

1 participant