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
Bump jedi to at least 0.16.0 and fix deprecated function usage #12207
Conversation
That sounds good ! Thanks ! |
would forcing support of jedi>=0.15.2 have been sufficient ? we're a bit stuck during the Spyder transition, as some fixes will only be in Jedi-0.17.1 |
Sorry I missed this question; I'll see in the backport if I can pin to only 0.15.2, but it's less than a month difference, so it won't change much I think. And so far this is only planned to be released in 8.0 which is not before fall I think. |
@meeseeksdev backport to 7.x |
…cated function usage
and no .complete() is only 0.16.0+ |
…207-on-7.x Backport PR #12207 on branch 7.x (Bump jedi to at least 0.16.0 and fix deprecated function usage)
…0 and fix deprecated function usage)"
…pr-12207-on-7.x Revert "Backport PR #12207 on branch 7.x (Bump jedi to at least 0.16.0 and fix deprecated function usage)"
This should make jedi 0.18 compatible with 7.x. Pinning is not an option as the resolver is free to pick older IPython with jedi 0.18 -- Bump jedi to at least 0.16.0 and fix deprecated function usage
Full sha of original commit: 5e979ec This should make jedi 0.18 compatible with 7.x. Pinning is not an option as the resolver is free to pick older IPython with jedi 0.18 -- Bump jedi to at least 0.16.0 and fix deprecated function usage
This should fix #12102
I ran
iptest IPython.core.tests.test_completer
and it didn't seem to give any errors. Not sure if we want to add any new tests here, or if that is good enough.According to jedi,
Multiple signatures are typical if you use Python stubs with @overload.
I made my change to account for all signatures, but not sure how this plays out IRL since I've never seen a function with multiple signatures.