Skip to content
This repository has been archived by the owner on Feb 12, 2022. It is now read-only.

Rest syntax #1987

Closed
Closed

Conversation

ManasJayanth
Copy link
Contributor

Release note: Adds support for rest syntax. Abstract Interpreter support pending.
Addresses #927

@ManasJayanth
Copy link
Contributor Author

ManasJayanth commented May 21, 2018

Flow keeps crashing with Flow_error.EMergeTimeout(105.016726017) on my mac. There already is an open issue

@ManasJayanth ManasJayanth changed the title [WIP] Rest syntax Rest syntax May 21, 2018
@hermanventer
Copy link
Contributor

It looks like the latest build has a real Flow issue.

@ManasJayanth ManasJayanth changed the title Rest syntax [WIP] Rest syntax May 23, 2018
@ManasJayanth ManasJayanth force-pushed the rest-syntax branch 3 times, most recently from 25a0358 to 631bda4 Compare May 28, 2018 19:57
@ManasJayanth
Copy link
Contributor Author

@hermanventer Can I please get some help with test262 tests? I'm not able to figure out which tests are failing because of this PR. I did spot some destructuring related failures earlier. Fixed them.

@ManasJayanth ManasJayanth changed the title [WIP] Rest syntax Rest syntax May 28, 2018
@hermanventer
Copy link
Contributor

Please rebase this when PR #2092 has been merged.

@trueadm
Copy link
Contributor

trueadm commented Jun 12, 2018

@prometheansacrifice Please can you merge/rebase master on this PR?

@ManasJayanth
Copy link
Contributor Author

@trueadm All tests pass :)

Copy link
Contributor

@trueadm trueadm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome work! Thank you.

Copy link

@facebook-github-bot facebook-github-bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@trueadm is landing this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

caiismyname pushed a commit that referenced this pull request Jun 22, 2018
Summary:
Release note: Adds support for rest syntax. Abstract Interpreter support pending.
Addresses #927
Closes #1987

Differential Revision: D8383089

Pulled By: trueadm

fbshipit-source-id: d58d3eb2296c7085d71afdf65ac7a553fc1d12e7
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants