Skip to content

Fix broken internal links and anchors - #310

Merged
cottage14 merged 1 commit into
apache:masterfrom
justinmclean:fix-broken-links
Oct 15, 2026
Merged

cottage14 merged 1 commit into
apache:masterfrom
justinmclean:fix-broken-links

Conversation

@justinmclean

Copy link
Copy Markdown
Member

Fixes 15 broken internal links found with a link checker: permalink anchors that did not match their heading ids, links missing # or https://, two wrong page names and two anchor typos. Five more need a decision on the right target and are not changed here: openpgp #ContinueAfterGeneration and #foaf, release-signing #basics and #faq, and services #public_html.

Correct 15 links: mismatched permalink anchors, missing # and https://, wrong page names and anchor typos.
@jbampton

Copy link
Copy Markdown
Member

It is great to fix links but you should really be adding a check or test to stop regressions.

Checkout some of the other ASF Projects some use lychee or markdown-link-check

https://github.com/lycheeverse/lychee

https://github.com/tcort/markdown-link-check

There are some ASF approved actions but it would be best to add a pre-commit config as I mentioned in #312 to enable local testing

https://github.com/apache/infrastructure-actions/blob/e3e0a2261a9c6d5bdd882802b303f34b1511b6df/approved_patterns.yml#L279

https://github.com/apache/infrastructure-actions/blob/e3e0a2261a9c6d5bdd882802b303f34b1511b6df/approved_patterns.yml#L380

@cottage14
cottage14 merged commit e33fe84 into apache:master Oct 15, 2026
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