Disable language services if Pyrefly extension installed + active#24987
Merged
karthiknadig merged 11 commits intomicrosoft:mainfrom Apr 23, 2025
Merged
Disable language services if Pyrefly extension installed + active#24987karthiknadig merged 11 commits intomicrosoft:mainfrom
karthiknadig merged 11 commits intomicrosoft:mainfrom
Conversation
Summary: Test Plan: Reviewers: Subscribers: Tasks: Tags:
rchiodo
reviewed
Apr 16, 2025
rchiodo
reviewed
Apr 16, 2025
Author
|
Thanks for the quick review @rchiodo! I did not realize you would see draft PRs yet... I just updated the summary so it should make more sense. |
rchiodo
reviewed
Apr 16, 2025
|
When you did this part:
Did you clone into the extensions folder for vscode? I think that's what those directions are asking you to do? @karthiknadig would know more. Pylance would have to be installed in the extensions folder I think for your testing to work |
|
This error: Usually happens when pyright and pylance are enabled at the same time. It might also happen if pylance is started twice for some reason. |
Member
|
@kinto0 Please look into and accept the CLA. Otherwise we can't merge this. |
Summary: Test Plan: Reviewers: Subscribers: Tasks: Tags:
Summary: Test Plan: Reviewers: Subscribers: Tasks: Tags:
Author
@microsoft-github-policy-service agree [company="meta"] |
Author
@microsoft-github-policy-service agree company="meta" |
rchiodo
reviewed
Apr 18, 2025
rchiodo
previously approved these changes
Apr 18, 2025
Crazy40528
reviewed
Apr 21, 2025
Crazy40528
left a comment
Crazy40528
left a comment
There was a problem hiding this comment.
kinto0:kinto/disable-with-pyrefly
Crazy40528
reviewed
Apr 21, 2025
Crazy40528
reviewed
Apr 21, 2025
karthiknadig
requested changes
Apr 21, 2025
Summary: Test Plan: Reviewers: Subscribers: Tasks: Tags:
…-python into kinto/disable-with-pyrefly
Summary: Test Plan: Reviewers: Subscribers: Tasks: Tags:
karthiknadig
approved these changes
Apr 22, 2025
DonJayamanne
approved these changes
Apr 22, 2025
facebook-github-bot
pushed a commit
to facebook/pyrefly
that referenced
this pull request
May 7, 2025
…d on + vice-versa) Summary: ms-python insiders came out last night and it is not looking good for my [patch](microsoft/vscode-python#24987). When I put up the PR, I was not able to test pylance fully and put too much faith in the test framework (it's really good but not this good apparently). It works great on a window reload, but that isn't good enough for testing. two issues: - installing pyrefly refreshes something but does not keep pylance disabled (green = pylance syntax highlighting) {F1977750850} I'm not sure what causes this. these settings are [re-checked on any extension install](https://github.com/microsoft/vscode-python/blob/a3dd3aa1bca82be1fb5c44f04c689233010eaeab/src/client/languageServer/watcher.ts#L345) (hence the flicker) and we [check for existence of pyrefly](https://github.com/microsoft/vscode-python/blob/a3dd3aa1bca82be1fb5c44f04c689233010eaeab/src/client/common/configSettings.ts#L283). - changing the disableLanguageServices setting does not actually trigger the change {F1977750864} This is an easier fix: I must've messed up testing it [here](microsoft/vscode-python#24987 (comment)) In the meantime, this hack fixes both of these issues from within pyrefly with no side effects, but only if a workspace is opened (it does not work on the default workspace [because settings are not watched there](https://github.com/microsoft/vscode-python/blob/a3dd3aa1bca82be1fb5c44f04c689233010eaeab/src/client/languageServer/watcher.ts#L295)) Reviewed By: SamChou19815 Differential Revision: D74327314 fbshipit-source-id: 7d97476572731e8771af44af60c81d0d2334054d
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
For #24850
Summary:
Background:
A new typechecker called Pyrefly will be featured at Pycon with a talk, website/sandbox (still WIP), and extension (still WIP). This extension will provide ultrafast typechecking and language services. When the Pyrefly extension is installed,
ms-python.pythonshould not start Jedi or Pylance unlesspython.pyrefly.disableLanguageServicesis set totrue.Test Plan:
https://github.com/user-attachments/assets/395bacbb-7ad0-4357-b084-cd5e88062801