Skip to content
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

Melange playground #758

Open
8 tasks
davesnx opened this issue Aug 29, 2023 · 4 comments
Open
8 tasks

Melange playground #758

davesnx opened this issue Aug 29, 2023 · 4 comments

Comments

@davesnx
Copy link
Member

davesnx commented Aug 29, 2023

Since Melange released a new playground we could integrate it with the current ReasonML website, either with a redirect or embed it directly.

JavaScript and Native compilations are topics that are spreaded across the website, and it's a must fix. Either of the solution we choose, would be nice to have it clear for users to understand Reason + Melange -> JS, Reason + dune -> native.

This issue will be used as an umbrella for all issues related with current Playground:

@davesnx
Copy link
Member Author

davesnx commented Aug 29, 2023

Preferred solution would be to showcase both playgrounds: sketch.sh and melange.re as a page.

and remove/deprecated the current playground/refmt.js

@jchavarri
Copy link
Contributor

ftr Sketch-sh/sketch-sh#371.

@jchavarri
Copy link
Contributor

I just tried in the melange playground all the referred issues that were crashing in the reason playground, and all seem to work fine now:

Issue Code Melange playground link
#604 type x = A{x: int}; Link
#607 type t = | Hi({name: string}); Link
#544 Js.log(0x lxor 0x80000000);Js.log(19 lxor 0x40000000); Link
#661 let a : int = 1s; Link

Maybe we can close those with a note directly?

@davesnx
Copy link
Member Author

davesnx commented Aug 29, 2023

Once we fix this issue, totally!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants