-
Notifications
You must be signed in to change notification settings - Fork 92
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
rename positron/positron_ipykernel
to posit/positron
#6085
Conversation
E2E Tests 🚀 |
@@ -43,6 +46,19 @@ | |||
|
|||
_sentinel = object() | |||
|
|||
# update Jedi cache to not conflict with other Jedi instances | |||
if platform.system().lower() == "windows": |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Learnt from prior changes in this module, if we do not update the cache we will not get any completions (slack conversation)
Adapted from jedi.settings.cache_directory
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great catch!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1 -- if this becomes an issue again, I suppose the kernel could also be promoted out of extensions/positron-python into the top-level of the Positron project
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks!!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
@@ -43,6 +46,19 @@ | |||
|
|||
_sentinel = object() | |||
|
|||
# update Jedi cache to not conflict with other Jedi instances | |||
if platform.system().lower() == "windows": |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great catch!
Release Notes
New Features
Bug Fixes
positron-python
paths are too long in Windows Error during user setup installation #4387QA Notes
should pass all tests. you should also still be able to run a local build and the LSP works.