-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Introduce syntax highlighting #1140
Conversation
Add code syntax highlighting
This pull request is being automatically deployed with Vercel (learn more). 🔍 Inspect: https://vercel.com/mdx/mdx/n2t5ij3rd |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me 💯
Just checking if I need to try and resolve the CI errors for this to merge, as I don't believe the issues relate to the patch I have proposed? |
@samrobbins85 There are two lint errors introduced by your PR. (Ignore the remark warnings, they are tracking at mdx-js/eslint-mdx#217) |
Fix eslint errors
* Update code-block.js Add code syntax highlighting * Update code-block.js Fix eslint errors * Second fix of eslint errors
* Update code-block.js Add code syntax highlighting * Update code-block.js Fix eslint errors * Second fix of eslint errors
Add code syntax highlighting
Before:
After: