-
Notifications
You must be signed in to change notification settings - Fork 17
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
Does not seems to work with flow version 0.17.0 #41
Comments
I've briefly looked into and it seems like error is coming from flow itself, more specifically here is some of my observations: I set a breakpoint on the above mentioned line to try to figure what is going on and found out following: So I tried running same from the terminal: /usr/local/bin/flow type-at-pos 22 48 --json --path /Users/gozala/Projects/reflex-virtual-dom-renderer/src/node.js That keeps hanging while this produces similar error as I see in terminal: /usr/local/bin/flow type-at-pos 22 48 /Users/gozala/Projects/reflex-virtual-dom-renderer/src/node.js --json
File not found While following seems to work fine: /usr/local/bin/flow type-at-pos /Users/gozala/Projects/reflex-virtual-dom-renderer/src/node.js 22 48 --json
{"type":"(unknown)","reasons":[],"path":"","line":0,"endline":0,"start":1,"end":0} My guess is something else deals with |
Ok so I have tried changing following line https://github.com/lukehoban/atom-ide-flow/blob/master/lib/util-flow-command.coffee#L79 to:
Which does not seem to resolve it. From web inspector I can now get a command to run in my terminal: getFlowCommand() + ' ' + args.join(' ') // => /usr/local/bin/flow type-at-pos /Users/gozala/Projects/reflex-virtual-dom-renderer/src/node.js 14 36 --json and running it in terminal produces following: /usr/local/bin/flow type-at-pos /Users/gozala/Projects/reflex-virtual-dom-renderer/src/node.js 14 36 --json
{"type":"(unknown)","reasons":[],"path":"","line":0,"endline":0,"start":1,"end":0} |
So I'm clueless to what is going on, only other thing worth noting is that I have flow installed through readlink /usr/local/bin/flow
../lib/node_modules/flow-bin/cli.js But updating flow path to an actual file still does not seem to work so I'm giving up. |
After digging further into it I realized that flow-bin is node-wrapper that uses actual Give that |
BTW changes I made originally did not turn out to be necessary after fixing the path. |
For people who stumble across this issue I had a similar initial error |
Running this command worked for me |
running type check produces following error in the web-inspector console:
The text was updated successfully, but these errors were encountered: