Skip to content

Streamline JS method chaining examples #457

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

Merged
merged 1 commit into from
Jan 4, 2022

Conversation

fhammerschmidt
Copy link
Member

@fhammerschmidt fhammerschmidt commented Oct 29, 2021

This example from the pipe docs is weird, its predecessor uses Js.Array2, while this one relies on extra bindings to map and filter in the prelude.

I deleted the superfluous bindings and used Js.Array2 as well here, instead.

@ryyppy
Copy link
Member

ryyppy commented Jan 4, 2022

good change. Thanks!

@ryyppy ryyppy merged commit f1503c3 into rescript-lang:master Jan 4, 2022
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.

2 participants