Skip to content

Conversation

@nini-faroux
Copy link
Contributor

No description provided.

Copy link
Contributor

@bladyjoker bladyjoker left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks Nigel! I appreicate this!

If you have 5-10 minutes to find all the links like this one (../smtn) and fix them the same way that would be greatly appreciated!

@bladyjoker bladyjoker added the documentation Improvements or additions to documentation label Nov 21, 2023
@nini-faroux
Copy link
Contributor Author

Sure no worries, I'll do that in a moment

@nini-faroux
Copy link
Contributor Author

Bah, these are broken too as I'm missing the tree/main part of the url, I'll fix that now

@bladyjoker
Copy link
Contributor

It's sad that mdbook doesn't check for link validity.

@nini-faroux
Copy link
Contributor Author

Ah that would be handy for sure - okay I think that's all of them, or most of them at least, if I spot any more as I go along I'll add them too

@bladyjoker
Copy link
Contributor

Lol https://github.com/Michael-F-Bryan/mdbook-linkcheck#configuration, I'll integrate this eventually! Thank you @nini-faroux

@nini-faroux
Copy link
Contributor Author

No worries!

@bladyjoker bladyjoker added this pull request to the merge queue Nov 21, 2023
Merged via the queue into main with commit 2e3f7df Nov 21, 2023
@bladyjoker bladyjoker deleted the nf/broken-link branch November 21, 2023 17:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants