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

First draft implementation of unite source for manpages #2

Closed
wants to merge 3 commits into from

Conversation

andreicristianpetcu
Copy link

I made a pull request for issue #1

@andreicristianpetcu
Copy link
Author

install unite.vim and run
:Unite manpage -start-insert

@jez
Copy link
Owner

jez commented Dec 29, 2014

As far as I can tell, everything looks good here. That being said, the central functionality of this plugin is to provide a command line wrapper around the $VIMRUNTIME/ftplugin/man.vim plugin bundled with Vim itself. Since I don't personally use Unite.vim, I'm wary of merging your pull request. I wouldn't be a very good maintainer. Is there a reason why this can't just be made into a standalone plugin?

@andreicristianpetcu
Copy link
Author

I was hoping that instead of having several plugins that focus on reading man pages, it would be a one-stop-shop for this. I will make it intro a separate plugin if you wish.

Check out Unite.vim (check out my dotfiles to see what other plugins play well with Unite.vim). If it were one plugin that I recommend it would be Unite.vim. Somehow (probably by magic) unite outline show an outline of the man pages https://github.com/andreicristianpetcu/dotfiles/blob/c76e32a0fb4c41e4917c9af50ed495c01ec9a471/vimrc#L330-L332

@jez
Copy link
Owner

jez commented Dec 31, 2014

I think I'd rather have it in another plugin. If you do end up writing it, I'd love to link to it from here in the README, so let me know!

@jez jez closed this Dec 31, 2014
@jez jez mentioned this pull request Dec 31, 2014
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.

None yet

2 participants