Skip to content

CI: revert the warning change once IPython updates #31407

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

Closed
charlesdong1991 opened this issue Jan 28, 2020 · 4 comments
Closed

CI: revert the warning change once IPython updates #31407

charlesdong1991 opened this issue Jan 28, 2020 · 4 comments
Labels
CI Continuous Integration

Comments

@charlesdong1991
Copy link
Member

A couple days ago, there was a CI broken caused by the latest jedi release, and back then, made a patch to catch the warning. Ideally, after ipython updates and fixes the deprecated code, then the warning should be removed, and we should be able to revert the change in #31323

FYI, I created an issue on IPython: ipython/ipython#12102

@jbrockmendel
Copy link
Member

@charlesdong1991 how will we know when this is ready to address?

@charlesdong1991
Copy link
Member Author

@jbrockmendel

i think we can already address this, I just checked the issue ipython/ipython#12102 and seems it has been solved and the new release of ipython came out a couple weeks ago.

Pls let me revert the original PR and see if anything breaks.

@jbrockmendel
Copy link
Member

@charlesdong1991 some tab completion tests are failing, is that related to this?

@mroeschke
Copy link
Member

Looks like the warning code has been reverted on master for quite some time since 2021-02-02, so appears safe to close.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI Continuous Integration
Projects
None yet
3 participants