Skip to content

Add some language identifier to enable syntax highlighting #250

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Sep 22, 2023

Conversation

sachaos
Copy link
Contributor

@sachaos sachaos commented Sep 20, 2023

I've added some language identifiers to code block for enabling syntax highlighting. :)

@ricochet
Copy link
Contributor

Thank you for adding this!

We have a little bit of a chicken and egg problem where WIT is defined by this repo but the grammar is specified in bytecodealliance/vscode-wit. The linguist tool used by GitHub for code blocks rendering in md pulls the grammar from that repo. I think this can be OK so long as it is clear that there will be a lag between incoming changes to WIT syntax and when rendering for the new syntax is supported.

I noticed that this change does not add language identifiers for all code blocks, e.g. there are many more blocks in WIT.md.

@ricochet ricochet requested a review from lukewagner September 22, 2023 14:24
@lukewagner
Copy link
Member

lgtm, thanks!

@lukewagner lukewagner merged commit cdae165 into WebAssembly:main Sep 22, 2023
@sachaos sachaos deleted the add-some-language-identifier branch September 22, 2023 16:16
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