Skip to content
This repository has been archived by the owner on Jan 10, 2024. It is now read-only.

Flow typedef doesn't work? #30

Closed
ajhyndman opened this issue Aug 30, 2017 · 3 comments
Closed

Flow typedef doesn't work? #30

ajhyndman opened this issue Aug 30, 2017 · 3 comments

Comments

@ajhyndman
Copy link

https://github.com/facebookincubator/idx/blob/master/packages/idx/src/idx.js.flow#L12

Is there a version of flow which comes with this type baked in? For open-source users, it'd be nice if the distributed typedef was useful.

@zertosh
Copy link
Member

zertosh commented Aug 31, 2017

$Facebookism$Idx is already baked into Flow. The idx.js.flow is just a convenience wrapper around it so that you automatically get it when you do import idx from 'idx';.

See https://flow.org/try/#0CYUwxgNghgTiAEA3W8CWwAeAueASAYlGCAEYD2ZA1qgM4C2uAkpgNwBQbALgJ4AOCAVRogY8ALzwA3m3jwArsJg4A-NNmyAdlDogcNTjFQaA5gBoZ6gGaGQG4DRUBBGDCjcAPEJEA+c7IC+bP7sbKCQsAjIorwwZLwO8F4wIWBkGvrwWjriaJgAFDFxNKbwAPri3mUAdAoiVVkgAJQheQ16BkbGzRyt2rrwyvqGJt1AA

@yungsters
Copy link
Contributor

Versions of Flow starting with the one that was release when idx was initially made available to the public should all support $Facebookism$Idx.

@ajhyndman
Copy link
Author

ajhyndman commented Aug 31, 2017

It is built into flow? Thanks for the explanation.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants