Skip to content

Conversation

@hackervole
Copy link

Summary

None

Purpose of change

Started out fixing a couple of broken links under /doc/c++ that I ran into while trying to set up a build environment, then I found a nifty tool to automatically check for broken Markdown links and got ambitious.

Describe the solution

Fixes pretty much all the broken links I found. There are probably still some broken heading links since I didn't check all of those, and there was at least one reference to a no-longer-existing data file that I wasn't sure of an appropriate replacement for.

Describe alternatives you've considered

N/A

Testing

Tested all the links in my local copy as I was editing them, and also checked some of them again on GitHub after pushing to my fork.

Additional context

Mostly from whenever things were moved from COMPILING/ to c++/, but a
few other random ones.
It seems that `placeholder.com` no longer exists, but I found
`placehold.co` which seems to do about the same thing. Using `?text=+`
didn't seem to work, so I used `@` instead.
There may be a few heading links that are still broken, but this should
be most of them.
@github-actions github-actions bot added <Documentation> Design documents, internal info, guides and help. NPC / Factions NPCs, AI, Speech, Factions, Ownership Map / Mapgen Overmap, Mapgen, Map extras, Map display [Markdown] Markdown issues and PRs new contributor json-styled JSON lint passed, label assigned by github actions astyled astyled PR, label is assigned by github actions labels Oct 31, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

astyled astyled PR, label is assigned by github actions <Documentation> Design documents, internal info, guides and help. json-styled JSON lint passed, label assigned by github actions Map / Mapgen Overmap, Mapgen, Map extras, Map display [Markdown] Markdown issues and PRs new contributor NPC / Factions NPCs, AI, Speech, Factions, Ownership

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant