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

Update documentation for Go-Guru switch #7247

Closed
wants to merge 3 commits into from

Conversation

jredville
Copy link
Contributor

This branch is based off of @edrex's work on switching the go layer to use go-guru (#6774). It updates the documentation to warn about Oracle's deprecation and a few usage notes on go-guru.

@jredville
Copy link
Contributor Author

@bogdanteleaga any feedback? Trying to get @edrex's work moved forward

Copy link

@nilbot nilbot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Go Oracle has been deprecated as of October 1, 2016, it's replacement is =go-guru=.

If you would like to use the =goguru= bindings in your work, you will need to
install it, and in your project you will need to set the scope with =SPC m f o=.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Keybindings should be surrounded with ~.

@d12frosted
Copy link
Contributor

It's not critical, but it would be nice if you fix your history, so it doesn't have that merge commit. It doesn't block me from merging (because I will simply cherry-pick two commits), but at least in future try to avoid such commits 😸

edrex and others added 3 commits October 14, 2016 10:13
change to prefix for guru commands from `mr` to `mf`
to separate from rename commands. Better prefix?

Fixes syl20bnr#6772.
@jredville
Copy link
Contributor Author

Updated, and will avoid in the future

@JAremko
Copy link
Collaborator

JAremko commented Oct 15, 2016

@d12frosted @TheBB :shipit:

@TheBB
Copy link
Contributor

TheBB commented Oct 17, 2016

Thanks! Squashed your two commits and cherry-picked in develop. You can safely delete your branch.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants