Skip to content

Conversation

@tsnobip
Copy link
Member

@tsnobip tsnobip commented Nov 21, 2025

No description provided.

@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Nov 21, 2025

Deploying rescript-lang-org with  Cloudflare Pages  Cloudflare Pages

Latest commit: 655d09b
Status: ✅  Deploy successful!
Preview URL: https://4771e154.rescript-lang.pages.dev
Branch Preview URL: https://upgrade-codemirror-to-v6.rescript-lang.pages.dev

View logs

@fhammerschmidt
Copy link
Member

Please restore at least highlighting of < >, </ and />.
If of course it's trivial to highlight the whole JSX tag then that would be even better.

@fhammerschmidt
Copy link
Member

The and keyword and the new triple operators also need some love:
image

@fhammerschmidt
Copy link
Member

same goes for async/await
image

@tsnobip tsnobip force-pushed the upgrade-codemirror-to-v6 branch from d3f9eef to c544096 Compare November 24, 2025 08:28
@tsnobip tsnobip force-pushed the upgrade-codemirror-to-v6 branch from 1a52c5c to cdcfa34 Compare November 24, 2025 09:21
@tsnobip
Copy link
Member Author

tsnobip commented Nov 24, 2025

@fhammerschmidt, I addressed all your issues, I also highlighted JSX attributes and function parameters, let me know what you think, I can easily color them back to white if you prefer.

@fhammerschmidt
Copy link
Member

Nice! LGTM!

@tsnobip tsnobip merged commit 90119ae into master Nov 24, 2025
2 checks passed
@tsnobip tsnobip deleted the upgrade-codemirror-to-v6 branch November 24, 2025 14:58
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.

4 participants