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

updated vim-ipython (pending) #1120

Closed
wants to merge 1 commit into from
Closed

Conversation

ivanov
Copy link
Member

@ivanov ivanov commented Dec 7, 2011

Just wanted to place this in the queue to make sure that an updated version goes into 0.12

I wanted to add this but not have it merged until 0.12 RCs are being cut, but in case I don't get to it again, this version is a big improvement over what ships with ipython currently.

ivanov/vim-ipython 3b62ca30c4cd3912532855279cdd161d2ef3c640
@fperez
Copy link
Member

fperez commented Dec 13, 2011

Does this need any further updates, or should we just merge it as-is? I think we're about ready to cut the first RC in a day or two.

@fperez
Copy link
Member

fperez commented Dec 13, 2011

@ivanov: last call... Do you want us to wait on this one for further updates, or should we go ahead and merge? I'd like to get to the RC pretty soon...

@ivanov
Copy link
Member Author

ivanov commented Dec 13, 2011

sorry, I've been a bit occupied - please merge as is

@fperez
Copy link
Member

fperez commented Dec 14, 2011

Was it you taking the pic or holding the sign?

@ivanov ivanov closed this in 633cbe1 Dec 14, 2011
@fperez
Copy link
Member

fperez commented Dec 14, 2011

OK, merged and pushed. Thanks!

@ivanov
Copy link
Member Author

ivanov commented Dec 14, 2011

It was me holding the sign - I know it's not PEP-8 compliant - I'll pylint and pyflakes and otherwise PEP8-proof my future political activity.

@fperez
Copy link
Member

fperez commented Dec 14, 2011

Otherwise we won't commit! :)

mattvonrocketstein pushed a commit to mattvonrocketstein/ipython that referenced this pull request Nov 3, 2014
Various small updates to vim integration.

Closes ipython#1120, rebased to avoid unnecessary recursive merge.
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