search

tptee / FlowIDE

IDE-caliber support for Flow in Sublime Text
MIT License
38 stars 6 forks source link

Add several flow features and bugfixes #10

Closed davidswinegar closed 7 years ago

davidswinegar commented 7 years ago

Add coverage checking for Flow, some bugfixes and extra error handling. Also add two new features - the option to mix sublime and flow autocomplete, and to omit function parameters for autocomplete (similar to Nuclide). Add the ability to override settings on a per-project basis under the "FlowIDE" key, so that different projects can use different paths for Flow.

Also add an MIT license (discussed over email).

AriaMinaei commented 7 years ago

I just tried this branch. Fixes the two biggest painpoints. I'm very grateful.

I'd say the only thing remaining to bring FlowIDE up to par with Nuclide is the ability to show the flow errors in a small popup box. Because as soon as an error message gets a bit long, I'd have to run $ flow inside the terminal to really see what's going on. Afaik this feature is doable because Sublime added a capability that allows packages to do simple HTML rendering a while ago.

STRML commented 7 years ago

:+1: Amazing. The coverage output is exactly what I wanted (barely visible but there), and the autocomplete fixes are really great. Really nice work!

davidswinegar commented 7 years ago

Hey @tptee - would you be able to take a look at this?

tptee commented 7 years ago

Looking now!

tptee commented 7 years ago

Sorry for the wait, I went down a refactoring rabbit hole 😬 v1.2.0 is published and Package Control should pick up the update within the next 30 mins.