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

Renaming done to close #76 #86

Closed
wants to merge 3 commits into from
Closed

Conversation

cub-uanic
Copy link

No description provided.

@@ -9,10 +9,10 @@
1. Setup [Vundle]:

```
$ git clone http://github.com/gmarik/vundle.git ~/.vim/bundle/vundle
$ git clone http://github.com/gmarik/vundle.git ~/.vim/vundle/vundle
Copy link
Contributor

Choose a reason for hiding this comment

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

To avoid confusion we could use ~/.vim/vundles/vundle instead

@gmarik
Copy link
Contributor

gmarik commented Sep 4, 2011

Hey! Good job! )
Couple notes though, as it's not yet ready to go into master:

  1. we want to keep backward commands compatibility for a while(say till 2.0) otherwise many ppl will get pissed off after this update
  2. Documentation should use Vundle everywhere so new users start use new Commands
  3. Put BIG NOTE in readme saying about transition
  4. we should probably finally apply @0's Allow comments after Bundle declarations #49 along the way

@0
Copy link
Contributor

0 commented Sep 4, 2011

If you're going to keep :Bundle around for backwards compatibility, you might as well make the new :Vundle with -bar as in #49, and deprecate :Bundle without changing it.

@gmarik
Copy link
Contributor

gmarik commented Sep 4, 2011

Very good point @0!

@cub-uanic
Copy link
Author

I'm not a vim-script wizard. All what I did - it's
%s/Bundle/Vundle/g
%s/bundle/vundle/g
through all files, and then revert several exceptions. If you don't mind, I prefer to leave polishing it to you.
Thanks for great plugin!

@gmarik
Copy link
Contributor

gmarik commented Sep 5, 2011

@cub-uanic, sure, no problem!
thank you

@martinklepsch
Copy link

Is there anything else that needs to be done in order to get this to work?

@greduan
Copy link

greduan commented Dec 2, 2012

@gmarik , as @mklappstuhl said.

What's holding you from implementing this?

@jdevera
Copy link
Contributor

jdevera commented Apr 4, 2014

#76 is closed now, see PR #403

@jdevera jdevera closed this Apr 4, 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.

6 participants