Skip to content

Fix backward compatibility and tests - #525

Merged
rmorshea merged 3 commits into
ipython:masterfrom
Jamim:fix/tests
Jul 5, 2019
Merged

rmorshea merged 3 commits into
ipython:masterfrom
Jamim:fix/tests

Conversation

@Jamim

@Jamim Jamim commented Jun 22, 2019

Copy link
Copy Markdown
Contributor

Hello,

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

Thanks!

Jamim added 3 commits June 22, 2019 02:25
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

Jamim commented Jun 24, 2019

Copy link
Copy Markdown
Contributor Author

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

@Jamim

Jamim commented Jul 2, 2019

Copy link
Copy Markdown
Contributor Author

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

@rmorshea

rmorshea commented Jul 5, 2019

Copy link
Copy Markdown
Contributor

@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

rmorshea commented Jul 5, 2019

Copy link
Copy Markdown
Contributor

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

Carreau commented Jun 4, 2020

Copy link
Copy Markdown
Member

"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