Closed xaviervia closed 8 years ago
@pirelenito @Deschtex the situation right now is:
babel-eslint
as the parser makes standard
play nicely with flowtype annotations, butI'm not sure this is a dealbreaker. Overall my idea was to add flowtype without causing any issues for people not intending to use it, and that's pretty much solved. I don't know if having linting of flow annotations is important for first drop. I checked out the two projects for adding linting rules for it and it doesn't seem that either of those is mature or bug free.
If you agree that this is not a dealbreaker I'll clean up and remove the WIP mark.
I think that should be fine! Amazing work :)
Stupid question about the linting... do we need to make any change on saguijs/eslint-config-sagui ?
@pirelenito yes we do. I'll PR that next.
This is WIP just because the eslint-plugin-flowtype works for ignoring the flowtype checks, but doesn't lint the flow related syntax. I'd like to figure out a way of making that work before merging.