-
Notifications
You must be signed in to change notification settings - Fork 28
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
Merge reasonable pull requests as much as possible #3
Comments
Sounds good @simnalamburt I'll get my project up to date with yours and submit a PR soonish. |
Thank you very much! |
I like slj#27 (plus they include doc updates in the PR, which is nice :)). Sound good? |
Sounds not bad. Looks like we need some rebase process. |
For...j3master to rebase against our fork? (sorry little confused about what you meant). In the meantime I've added j3master's fork as a remote and merged it into a new branch to make this PR: Could you take a peak when you get a chance - I played with a couple combinations of settings and it appears to work as advertized. I do like it even. |
Sorry for confusion, you did what I wanted |
Fixes: simnalamburt#3 Closes: sjl/gundo.vim#28 Regression: 3b94d2d Introduced in: 32142a6
The text was updated successfully, but these errors were encountered: