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 backward compatibility and tests #525

Merged
merged 3 commits into from
Jul 5, 2019
Merged

Conversation

Jamim
Copy link
Contributor

@Jamim Jamim commented Jun 22, 2019

Hello,

I'd like to propose some fixes.
Hope you find them useful.

Thanks!

Currently it's difficult to figure out which
versions of packages were actually installed.
nbconvert package on PyPI is out of date.
It causes test failure on CI.
I suppose that it makes sense to install nbconvert
from GitHub instead of PyPI.
Please look at a failure on CI:
travis-ci.org/ipython/traitlets/jobs/545082512
@Jamim
Copy link
Contributor Author

Jamim commented Jun 24, 2019

Hi @minrk,
Could you please take a look at this PR?
Thanks!

@Jamim
Copy link
Contributor Author

Jamim commented Jul 2, 2019

Hello @rmorshea,
Can you please take a look at this PR?
Thank you!

@rmorshea
Copy link
Contributor

rmorshea commented Jul 5, 2019

@Jamim I've been swamped recently. I may be able to take a look at this in the next few days.

@rmorshea rmorshea merged commit bb1cd9e into ipython:master Jul 5, 2019
@rmorshea
Copy link
Contributor

rmorshea commented Jul 5, 2019

Didn't realize the changes were so small, and it's great that this restores the tests. Thanks!

@Jamim Jamim deleted the fix/tests branch July 6, 2019 05:09
@Carreau Carreau added this to the 5.0 milestone Jun 4, 2020
@Carreau Carreau added 5.0-re-review Need to re-review for potential API impact changes. 5.0-minor rereviewed, minor change need to be put in changelog. labels Jun 4, 2020
@Carreau
Copy link
Member

Carreau commented Jun 4, 2020

"is deprecated in traitlets 4.4, and may cause "

I'm not sure there was or ever will be a 4.4 so may need to get updated.

@Carreau Carreau removed the 5.0-re-review Need to re-review for potential API impact changes. label Jun 22, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
5.0-minor rereviewed, minor change need to be put in changelog.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants