-
Notifications
You must be signed in to change notification settings - Fork 68
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
Add Flow annotations #11
Comments
👍 |
Why close this? I can't claim that it's a high priority for me, but I'd like to use Flow at some point. |
just figured no one is working on it and there is no point in keeping the issue around :) feel free to reopen. |
AFAIK nobody's working on any of the issues on the repo right now :D Gonna re-open this, thanks! |
I'm going to close this out again. Unfortunately Relay doesn't actually ship its Flow definitions, so I have no good way of making sure that any type checks I add here are consistent with what Relay has. I'll look at this again once the Flow ecosystem has a better story around packages that use Flow also shipping Flow type definitions, with an eye especially on how graphql/graphql-js#412 pans out. |
I think this is the next big thing we should add to this repo. Relay is fully typed with Flow and we should make that a standard for library around Flow.
The text was updated successfully, but these errors were encountered: