Skip to content
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

Fix argument passing and result parsing to work with new go-guru binary #39

Closed
wants to merge 1 commit into from
Closed

Conversation

smo921
Copy link

@smo921 smo921 commented Aug 11, 2016

  • Update cli args to work with new go-guru tool
  • There is no "what" section in the returned data, fix availableModes to account for that.

This fixes the problem reported in issue #20

…tion in the returned data, fix availableModes to account for that
@ig0rsky
Copy link

ig0rsky commented Aug 11, 2016

@smo921 you might want to publish this fork because it seems like this one is not maintained anymore :(

@smo921 smo921 mentioned this pull request Aug 11, 2016
@smo921
Copy link
Author

smo921 commented Aug 15, 2016

@NiggyWizzyWoz I am hoping to hear something about the PR before starting a new fork / atom plugin.

@nvartolomei
Copy link

@rubyist 🎱

@philoserf
Copy link

philoserf commented Oct 14, 2016

Is it time to fork to a go-guru plugin? See also issue #38 . No activity on GitHub from Rubyist since early August.

see also: https://github.com/corvinusy/my-go-guru
see also: https://github.com/smo921/go-oracle/pull/1
see also: https://github.com/serfdom/go-guru/tree/fork-to-go-guru

@smo921
Copy link
Author

smo921 commented Oct 14, 2016

I mentioned this in the Go slack channel and was pointed to this project. https://github.com/jnmoal/go-guru

I haven't tried it out yet.

@philoserf
Copy link

Thank you for the pointer to @jnmoal's package. It isn't working.

@jnmoal
Copy link

jnmoal commented Oct 18, 2016

Just to let you know, I'm still working on the go-guru package, I just don't have much time these days. I know it's not working, but I'm on it.

@smo921 smo921 closed this Feb 18, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants