-
Notifications
You must be signed in to change notification settings - Fork 26
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
typescript #37
Comments
something like this: https://github.com/pa-bru/graphql-cost-analysis/compare/master...sgeisbacher:feature/intro-typescript?expand=1 (first untested draft) |
Some time ago, I created types for it but I didn't publish them as I don't know if they are sufficient. You might want to use them. https://gist.github.com/ponjimon/0c085f624291d7575dc5666b71768ce4 |
@lookapanda Maybe you could create a PR for this? |
@Ponjimon the link gives 404, private gist? |
No, I just had a different username back then. I updated the link! |
are there any plans to switch from flow to typescript? i like your tool but its one of a few dependencies that neither has
@types/
-package nor has direct typescript-declaration in shipped npmjs.com-artifact. that prevents me from doing fully strict typescript-type-checking, cause such dependencies get implictAny type 😬The text was updated successfully, but these errors were encountered: