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

DOC: fixed using IP.prompt_manager which is removed from IPython 5.x #14004

Merged
merged 3 commits into from
Aug 18, 2016

Conversation

znmean
Copy link
Contributor

@znmean znmean commented Aug 15, 2016

this PR was made during PyCon APAC, pandas sprint

thanks

@znmean znmean changed the title fixed using IP.prompt_manager which is removed from IPython 5.x DOC: fixed using IP.prompt_manager which is removed from IPython 5.x Aug 15, 2016
@codecov-io
Copy link

codecov-io commented Aug 15, 2016

Current coverage is 85.28% (diff: 100%)

Merging #14004 into master will not change coverage

@@             master     #14004   diff @@
==========================================
  Files           139        139          
  Lines         50241      50241          
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
  Hits          42849      42849          
  Misses         7392       7392          
  Partials          0          0          

Powered by Codecov. Last update 6b7857b...cfd9499

reset the execution count if we haven't processed this doc
NOTE: this may be borked if there are multiple seen_doc tmp files
check time stamp?
"""
Copy link
Member

Choose a reason for hiding this comment

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

can you leave this as # comments? (that's how multiline comments in the codebase are done """ is used for docstrings)

@jorisvandenbossche jorisvandenbossche added Build Library building on various platforms Docs labels Aug 15, 2016
@jorisvandenbossche jorisvandenbossche added this to the 0.19.0 milestone Aug 15, 2016
@jorisvandenbossche
Copy link
Member

Thanks!
Added a small comment. I also think a whatsnew notice is not needed, as it is not a user facing change/bug fix

class promptManager has removed in IPython 5.x, but this file
still using that class, so it makes AttributeError.

close pandas-dev#14003
@znmean
Copy link
Contributor Author

znmean commented Aug 16, 2016

Thanks! Done what you said.

  • revert to # comment
  • no whatsnew notice

@jorisvandenbossche jorisvandenbossche merged commit 6d8044c into pandas-dev:master Aug 18, 2016
@jorisvandenbossche
Copy link
Member

@znmean Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Build Library building on various platforms Docs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

DOC: doc/make.py uses IPython promptManager, which is removed from IPython 5.x
3 participants