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

CI: Revert 31323 for deprecation warning from Jedi #35845

Closed
wants to merge 13 commits into from

Conversation

charlesdong1991
Copy link
Member

@charlesdong1991 charlesdong1991 commented Aug 21, 2020

ipython/ipython#12102 has been solved, and the new release came out a couple weeks ago, so revert should be fine according to ipython/ipython#12102 (comment)

@charlesdong1991 charlesdong1991 changed the title Revert 31323 CI: Revert 31323 Aug 21, 2020
@charlesdong1991 charlesdong1991 changed the title CI: Revert 31323 CI: Revert 31323 for deprecation warning from Jedi Aug 21, 2020
@charlesdong1991 charlesdong1991 marked this pull request as draft August 21, 2020 20:18
@charlesdong1991
Copy link
Member Author

emm, seems with the latest version of ipython (7.17.0) and jedi (0.17.2), the deprecation warning is still there.

I converted this PR to WIP, and will ask in ipython issue again about this.

@gfyoung gfyoung added the Dependencies Required and optional dependencies label Aug 23, 2020
@WillAyd
Copy link
Member

WillAyd commented Oct 23, 2020

Is this still active?

@charlesdong1991
Copy link
Member Author

emm, it is blocked by an ipython issue. my previous issue was closed ipython/ipython#12102 however, it wasn't being solved in the later version of ipython

there is a new issue ipython/ipython#12597 now tracking this warning issue, not sure if the fix will be included a newer version.

@jreback
Copy link
Contributor

jreback commented Nov 4, 2020

what is the status here?

@jreback
Copy link
Contributor

jreback commented Nov 26, 2020

@charlesdong1991 can you merge master and ping on green

@jreback
Copy link
Contributor

jreback commented Dec 29, 2020

we have a new jedi issue, but this one is moot.

@jreback jreback closed this Dec 29, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Dependencies Required and optional dependencies
Projects
None yet
Development

Successfully merging this pull request may close these issues.

CI: revert the warning change once IPython updates
5 participants