-
Notifications
You must be signed in to change notification settings - Fork 102
Add baselineMode extension setting for workspace for auto | discard
#1693
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
Add baselineMode extension setting for workspace for auto | discard
#1693
Conversation
DetachHead
left a comment
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 for the contribution!
i think this is something we can just document as a tip for users who want this behavior, rather than something integrated into the language server (i'm not sure how that would even work) |
This comment has been minimized.
This comment has been minimized.
Co-authored-by: DetachHead <[email protected]>
Agreed. It was more of a suggestion for users. I made a change to put the suggestion into the documentation (plus some other changes), let me know what you think. |
Co-authored-by: DetachHead <[email protected]>
|
According to mypy_primer, this change doesn't affect type check results on a corpus of open source code. ✅ |
Add a baselineMode extension setting at the workspace level for
auto | discard. Not includinglockor other settings for now since the behavior it would have in the extension is less clear to me, and--baselinemodeis referenced as experimental in the cli documentation. So it seems fine to be less aggressive with exposing modes to users here.Default behaviors are unchanged (still defaults to
auto).This satisfies the first part of the proposal in #1692.
Users using
discardin the extension may want to include a pre-commit hook running basedpyright with a different baselinemode (i.e.autoor no--baselinemode/--writebaselineflag) to get rid of baslined errors that are no longer relevant.This intentionally does not address #1524 yet, though there's no reason why it can't be included later.