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

Fix #1938 -- Jupyter compatibility #2178

Merged
merged 2 commits into from Nov 21, 2015
Merged

Fix #1938 -- Jupyter compatibility #2178

merged 2 commits into from Nov 21, 2015

Conversation

Kwpolska
Copy link
Member

This is #1938. cc @damianavila, @ltiao.

Signed-off-by: Chris Warrick <kwpolska@gmail.com>
@Kwpolska Kwpolska self-assigned this Nov 21, 2015
@Kwpolska Kwpolska added this to the v7.7.4 milestone Nov 21, 2015
@@ -7,7 +7,7 @@ pygal>=2.0.0
typogrify>=2.0.4
phpserialize>=1.3
webassets>=0.10.1
ipython[notebook]>=2.0.0
jupyter>=1.0.0
Copy link
Member

Choose a reason for hiding this comment

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

notebook should be the dependency... jupyter "metapackage" contain a lot of stuff which is not a dependency for the functionality...

Copy link
Member Author

Choose a reason for hiding this comment

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

Fixed.

@damianavila
Copy link
Member

Changes seems ok... I would say that sooner than later we need to deprecate the old API... but let's wait a little bit to do that...

Kwpolska added a commit that referenced this pull request Nov 21, 2015
@Kwpolska Kwpolska merged commit 00ebeab into master Nov 21, 2015
@Kwpolska Kwpolska deleted the jupyter branch December 24, 2015 10:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants