Skip to content
This repository has been archived by the owner on Dec 19, 2024. It is now read-only.

Check on edit #87

Merged
merged 4 commits into from
Mar 16, 2017
Merged

Check on edit #87

merged 4 commits into from
Mar 16, 2017

Conversation

Gozala
Copy link
Contributor

@Gozala Gozala commented Mar 15, 2017

Please note that this pull unfortunately includes changes from #85, I'll rebase this once hopefully #85 lands. This change implements short term solution described in #26. To be clear if you have edits in multiple files that aren't saved, it will report errors for the active document as if it was saved but will ignore all other unsaved changes because:

  1. Trying to type check as if all unsaved changes were saved isn't necessarily a desired or expected behavior. I personally like to make edits to files until it type checks and only then move on to other file. When I get blocked, I tend to move on to other file, with a plan to return to the blocker. With such work flow type checking all unsaved changes isn't a desired option.
  2. From what I understand currently it's not possible to do that anyway.

Gozala added 4 commits March 14, 2017 17:08
- `isWatching` got deprecated in favor of `isBackground`.
- latest vscode does not recognizes version `0.3.2`.
- Adds status indicator inspired by one in vscode-jest, that is shown spinner when flow is type busy.
- Adds `flow.showStatus` setting so that status indicator can be disabled if not desired.
@orta
Copy link
Contributor

orta commented Mar 15, 2017

This feels 👍 to me, as the commits form #85 are in here - I'll wait until that's merged for this to be good to go.

@orta
Copy link
Contributor

orta commented Mar 16, 2017

Alright, with #85 in, this looks good too 👍

@orta orta merged commit 63fecec into flow:master Mar 16, 2017
Gozala added a commit to Gozala/flow-for-vscode that referenced this pull request Mar 16, 2017
orta added a commit that referenced this pull request Mar 16, 2017
Describe changes from #85 #87 #88 in changelog
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants