Skip to content

Conversation

@BD103
Copy link
Member

@BD103 BD103 commented Sep 29, 2025

When trying to build the website, Zola emits the following warnings:

$ zola serve
Building site...
Warning: Highlight language text not found in learn/migration-guides/0.16-to-0.17.md
Warning: Highlight language text not found in learn/migration-guides/0.16-to-0.17.md
Warning: Highlight language text not found in learn/migration-guides/0.16-to-0.17.md
Warning: Highlight language text not found in learn/migration-guides/0.16-to-0.17.md
Warning: Highlight language text not found in learn/migration-guides/0.16-to-0.17.md
Warning: Highlight language raw not found in learn/migration-guides/0.16-to-0.17.md
Warning: Highlight language ron not found
Warning: Highlight language shell not found in learn/migration-guides/0.11-to-0.12.md
Warning: Highlight language shell not found in learn/migration-guides/0.14-to-0.15.md
Warning: Highlight language shell not found in learn/migration-guides/0.14-to-0.15.md
Warning: Highlight language shell not found in learn/migration-guides/0.14-to-0.15.md
Warning: Highlight language shell not found in learn/migration-guides/0.14-to-0.15.md
Warning: Highlight language ron not found in learn/migration-guides/0.14-to-0.15.md
Warning: Highlight language ron not found in learn/migration-guides/0.14-to-0.15.md

This PR fixes all of these warnings except for the ones about RON, which I've split out into a separate PR. I've added more detail to each individual commit, so I recommend reviewing them individually!

When you don't specify a language for a codeblock, it is assumed to be plain text, so we can just remove the `text` bit.
In this case, `raw` means plain text, so we can remove the attribute.
@BD103 BD103 added A-Migration Guides C-Correction A typo, problem or factual issue in what we've written S-Needs-Review D-Trivial labels Sep 29, 2025
@alice-i-cecile alice-i-cecile added S-Ready-For-Final-Review Ready for a maintainer to consider for merging and removed S-Needs-Review labels Sep 29, 2025
@alice-i-cecile
Copy link
Member

alice-i-cecile commented Sep 29, 2025

My gut reaction is that this will conflict with mdlint's preferences for labelling languages on every code block, but if CI is green I'm happy. We might need to disable a rule.

@alice-i-cecile alice-i-cecile added this pull request to the merge queue Sep 29, 2025
Merged via the queue into bevyengine:main with commit 66d3602 Sep 29, 2025
10 checks passed
@BD103 BD103 deleted the fix-highlight-warnings branch September 29, 2025 18:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-Migration Guides C-Correction A typo, problem or factual issue in what we've written D-Trivial S-Ready-For-Final-Review Ready for a maintainer to consider for merging

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants