mckinsey / vizro

Vizro is a toolkit for creating modular data visualization applications.
https://vizro.readthedocs.io/en/stable/
Apache License 2.0
2.46k stars 109 forks source link

[Docs] Make build actually strict #535

Closed antonymilne closed 1 week ago

antonymilne commented 1 week ago

Description

We run mkdocs build with --strict in order to catch errors like broken links, but this isn't doing anything because by default the messages are raised with level iNFO and not the required WARNING that would then be elevated to ERROR by --strict. See https://www.mkdocs.org/user-guide/configuration/#validation for explanation of these options.

mkdocs serve is just for development purposes so shouldn't fail with warnings, hence I removed --strict there.

Linkchecker still works as before, which is an external tool outside mkdocs and checks the built documentation by following external links. Since we now ignore links to our RTD docs themselves (see #524) it is essential we make our checks stricter so that we don't miss broken internal links within our own docs.

Changes to fix warnings include:

Notice

stichbury commented 1 week ago

I just looked at the new warnings. It shouldn't be too strict, and we should fix them. They also made it clear that there are a couple of broken links because of renamings of titles 😅

Maybe @stichbury can help you with this?

Sure, happy to fix these in the next sprint, just stick me on the ticket and I'll pick it up w/c 24th June.

huong-li-nguyen commented 1 week ago

Sure, happy to fix these in the next sprint, just stick me on the ticket and I'll pick it up w/c 24th June.

@stichbury - I hope you don't mind I took this over! I actually just wanted to check in this PR whether I've introduced any new broken anchor links with the recent PRs I've just merged. And it turned out that I did add some broken anchor links again 😅 It's very good that we have the strict check in place now!

When I started to fix them, I just fixed all of them 😄 We also plan to do the release on Monday, so probably better to have the docs fixed until then 👍

Feel free to review the changes though 🙏

stichbury commented 1 week ago

@stichbury - I hope you don't mind I took this over!

Of course not, and thank you 🌟 that's a big help and timesaver for me.