-
Notifications
You must be signed in to change notification settings - Fork 832
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 support #78
Comments
Yes. Really I am intent on support both flow and typescript. But unfortunatly I have really little time now, so the most relevant feature would be 24 hours, that I would work on. |
@dmtrKovalenko This is blocked by the deployment method. Since we don't distribute every component like material-ui and instead distribute a bundle of the three main components the typings won't be much of use. We should focus on the build process first and then worry about the typings. |
Ohh, I see. But its just like declarations for the file, it doesnt possible to declare all types in one file? |
@dmtrKovalenko It is possible, but not really recommended since it becomes hard to maintain. We should first decide on way we wish to deploy the package then write the typings. |
Hmm I`ll take a look on that, I think we should implement that anyway |
Greetins @alitaheri How is it going, are the typings already available? (I can't find them on npm 🤔 ) |
They are provided with the project directly. No need to install @types |
material-ui supports typescript. I think we should do the same. I'm willing to write and maintain the typings. Are you open to the idea? @dmtrKovalenko
The text was updated successfully, but these errors were encountered: