-
Notifications
You must be signed in to change notification settings - Fork 1.2k
Limit python.languageServer
setting scope to window
#13246
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
Conversation
Kudos, SonarCloud Quality Gate passed!
|
Codecov Report
@@ Coverage Diff @@
## master #13246 +/- ##
==========================================
+ Coverage 59.64% 59.80% +0.15%
==========================================
Files 670 670
Lines 36830 37235 +405
Branches 5188 5286 +98
==========================================
+ Hits 21968 22268 +300
- Misses 13770 13833 +63
- Partials 1092 1134 +42
Continue to review full report at Codecov.
|
I'm sure you tested it, but did you notice any oddities? I know we already have loads of settings set to "resource" that really shouldn't be, but are. |
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.
Tested it, seems to work fine in the expected cases.
Pylance does not support per-folder LS instances and
MPLS v1
does not reliably work in this case anyway. This simplifies transition of setting fromv1
toPylance
.- [ ] Has a news entry file (remember to thank yourself!).- [ ] Appropriate comments and documentation strings in the code.- [ ] Has sufficient logging.- [ ] Has telemetry for enhancements.- [ ] Unit tests & system/integration tests are added/updated.- [ ] Test plan is updated as appropriate.- [ ]package-lock.json
has been regenerated by runningnpm install
(if dependencies have changed).- [ ] The wiki is updated with any design decisions/details.