Skip to content

Added GitHub action to check links - #51

Draft
joachimmetz wants to merge 1 commit into
forensicswiki:mainfrom
joachimmetz:citests1
Draft

joachimmetz wants to merge 1 commit into
forensicswiki:mainfrom
joachimmetz:citests1

Conversation

@joachimmetz

@joachimmetz joachimmetz commented Dec 18, 2022 •

Copy link
Copy Markdown
Collaborator

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

joachimmetz commented Jan 1, 2023 •

Copy link
Copy Markdown
Collaborator Author

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

joachimmetz commented Jan 3, 2023 •

Copy link
Copy Markdown
Collaborator Author

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
joachimmetz force-pushed the citests1 branch 3 times, most recently from 934fe8c to 85c25e4 Compare January 12, 2023 02:33
@joachimmetz

Copy link
Copy Markdown
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

Copy link
Copy Markdown
Collaborator Author

Down to 54, some links cannot be correctly evaluated.

This branch has not been deployed

No deployments
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.

2 participants