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

Include the 'Belt' standard library #7

Open
austindd opened this issue Dec 4, 2019 · 3 comments
Open

Include the 'Belt' standard library #7

austindd opened this issue Dec 4, 2019 · 3 comments

Comments

@austindd
Copy link

austindd commented Dec 4, 2019

Since BuckleScript officially ships with Belt as an alternative standard library (and officially recommends its use), Belt should be available in ReBench.

I am not sure how to go about integrating it at this time, but it would be a very welcomed improvement!

@glennsl
Copy link
Member

glennsl commented Dec 19, 2019

This uses bs-in-a-box and is just a matter of replacing the vendored compiler and stdlib bundle there as long as the interface is still the same. Unfortunately I suspect it's not. I also think the process of building it has changed a lot.

I think @ryyppy and @thangngoc89 know the process best these days. Maybe they can chime in, or have plans to do something similar to this elsewhere.

@thangngoc89
Copy link

@glennsl I totally forgot about rebench, i’ll see if I can update this

@tsnobip
Copy link

tsnobip commented Jun 16, 2020

I agree, rebench is so useful to quickly benchmark BS code, having access to Belt would be a game changer because Belt implementation usually makes it much faster than the JS lib and a bit faster than Stdlib.

I don't know if things have become a bit easier to integrate since last year, I know there's some ongoing work with a new playground for reasonml.org, maybe some of the efforts could be reused here.

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

No branches or pull requests

4 participants