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

Make it clearer Opal is used on https://try.ruby-lang.org/ #122

Closed
eregon opened this issue Mar 6, 2022 · 2 comments · Fixed by #148
Closed

Make it clearer Opal is used on https://try.ruby-lang.org/ #122

eregon opened this issue Mar 6, 2022 · 2 comments · Fixed by #148

Comments

@eregon
Copy link
Member

eregon commented Mar 6, 2022

On https://try.ruby-lang.org/playground/ it is clear that Opal is the default, and one can choose CRuby.

But on the homepage, https://try.ruby-lang.org/ says nothing that Opal is used, and CRuby cannot be chosen.

This leads to confusion, e.g.:

I think the best would be for the homepage to also have the "engine selector" (Opal/CRuby), so the homepage would be like the Playground page (but still without the docs)

@eregon
Copy link
Member Author

eregon commented Mar 7, 2023

I believe this is a severe problem, people think https://try.ruby-lang.org/ is Ruby (= standard Ruby = CRuby) but it's Opal and it's incompatible in many ways. Here is a real life example which wasted hours of work due to this: oracle/truffleruby#2905 (comment)

I think the best way to change this is to default to CRuby, and barring that to make extremely clear it's Opal and not CRuby.
I have nothing against Opal, but this leads to a lot of confusion.

@eregon
Copy link
Member Author

eregon commented Mar 7, 2023

#134 seems another clear reason why Opal shouldn't be the default.
The Ruby programming language has a strong difference between Integer and Float.
Opal doesn't have that because JS doesn't have that, but the default playground should clearly respect this essential property of the Ruby language.

eregon added a commit to eregon/TryRuby that referenced this issue Mar 7, 2023
eregon added a commit to eregon/TryRuby that referenced this issue Jul 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging a pull request may close this issue.

1 participant