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

private fields and flow-bin@0.90.0 #7355

Closed
iztsv opened this issue Jan 15, 2019 · 4 comments
Closed

private fields and flow-bin@0.90.0 #7355

iztsv opened this issue Jan 15, 2019 · 4 comments
Labels

Comments

@iztsv
Copy link

iztsv commented Jan 15, 2019

Unfortunately, I was not able to reproduce the issue using https://flow.org/try

I've created the following repository: https://github.com/ilyaztsv/flow-0.90-private-fields

Please see README.md which describes how to reproduce the issue.

flow-bind@0.90.0 breaks using of private fields and I have the following error:

Error ┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈ src/index.js:0:1

Unexpected parser state: "Assert_failure src/parser_utils/signature_builder_generate.ml:653:58"

flow-bind@0.89.0 works as expected in my case.

Probably, it's related to my PC and environment. Let me know if you need more detailed information.

@jbrown215
Copy link
Contributor

Thanks for reporting!

@jbrown215 jbrown215 added the bug label Jan 15, 2019
@pakoito
Copy link
Contributor

pakoito commented Jan 15, 2019

Hi @ilyaztsv! Could you please share your .flowconfig? That'll help us find the issue.

@gabelevi
Copy link
Contributor

@pakoito - it's in the shared repository

facebook-github-bot pushed a commit that referenced this issue Jan 16, 2019
…te fields

Summary:
This diff prevents an exception reported in #7355 that'll cause Flow 0.9.0 to always except on private fields.

The fix is in place until we fully understand whether private properties are part of the generated signature or not.

This diff needs to be followed up in T39206072 after the release.

Reviewed By: panagosg7

Differential Revision: D13695528

fbshipit-source-id: 3c4ece37f8bc31da79c8c29ab365070270915756
@jbrown215
Copy link
Contributor

A fix is ready and will be shipped in 0.91, which is releasing this week.

jbrown215 pushed a commit that referenced this issue Jan 17, 2019
…te fields

Summary:
This diff prevents an exception reported in #7355 that'll cause Flow 0.9.0 to always except on private fields.

The fix is in place until we fully understand whether private properties are part of the generated signature or not.

This diff needs to be followed up in T39206072 after the release.

Reviewed By: panagosg7

Differential Revision: D13695528

fbshipit-source-id: 731378f3e364b8930ce8b7e80b98faa5d90836b3
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants