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

Escape the current path in keywordprg #32

Merged
merged 1 commit into from
Nov 22, 2017
Merged

Escape the current path in keywordprg #32

merged 1 commit into from
Nov 22, 2017

Conversation

andersonba
Copy link
Contributor

@andersonba andersonba commented Sep 20, 2015

Fix for current path escaping on OSX

Screenshot

@andersonba
Copy link
Contributor Author

@dag

@CamilleScholtz
Copy link

He doesn't seem to be active anymore, and there are still many PR open. Maybe someone should fork this project.

@lilyball
Copy link

There are 18 forks, but none of them are particularly active. Mine seems to have the most activity, but I haven't done anything with it in a while.

@CamilleScholtz
Copy link

I decided to fork this project myself and merged most of the PR's: https://github.com/onodera-punpun/vim-fish

@jsatk
Copy link
Contributor

jsatk commented Oct 18, 2016

Thank you @onodera-punpun! I switched to your fork. Issues resolved.

@jsatk
Copy link
Contributor

jsatk commented Jan 26, 2017

@onodera-punpun It appears you've deleted this fork. Switching back to main.

@CamilleScholtz
Copy link

@jsatk Yeah, I switched from vim to micro, and leaving an unmaintained fork up that intended to be maintained might confuse people.

@jsatk
Copy link
Contributor

jsatk commented Jan 27, 2017

@onodera-punpun thanks for the update! Any chance I could take ownership of your fork?

@CamilleScholtz
Copy link

@jsatk Yeah of course, I deleted the repo through so I don't know if (how?) that's possible? Anyways all I did was merge some PRs.

@jsatk
Copy link
Contributor

jsatk commented Jan 27, 2017

@onodera-punpun will do that. Thanks!

@jsatk
Copy link
Contributor

jsatk commented Apr 14, 2017

Can we close this PR?

@dag dag merged commit b1f7ca4 into dag:master Nov 22, 2017
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