search

stampit-org / stampit

OOP is better with stamps: Composable object factories.
https://stampit.js.org
MIT License
3.02k stars 102 forks source link

FlowType declarations #227

Closed koresar closed 1 year ago

koresar commented 8 years ago

@jmancine asked for a feature - FlowType declarations. Related FlowType docs are here.

danielkcz commented 8 years ago

Well I am not really sure what is currently better way of static type checking. I haven't get into any of those yet. From what I remember, when I tried flow last time, it wasn't working on Windows. I am not sure if that's still a true. Also what I don't like much about Flow is a need to add that /* flow */ comment there.

Lately I am hearing more and more about Typescript as a more robust solution. From what I seen out there, there is also ability to keep types in separate file. There is also the typings tool that can manage these files.

I am not sure if we should combine both of these mechanics really, might add uncessary complexity in maintenance.

koresar commented 8 years ago

They fixed flow type for Windows just yesterday AFAIK.

The /* flow */ comment is not necessary for FlowType declarations. Just like our TypeScript declarations.

We can support both declarations if anyone is willing to contribute.

PS: thanks for typings link!