Skip to content
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

Setting 'linting.lintOnTextChange' has been deprecated #313

Closed
DonJayamanne opened this issue Nov 28, 2017 · 0 comments · Fixed by #315
Closed

Setting 'linting.lintOnTextChange' has been deprecated #313

DonJayamanne opened this issue Nov 28, 2017 · 0 comments · Fixed by #315
Labels
area-linting feature-request Request for new features or functionality
Milestone

Comments

@DonJayamanne
Copy link

DonJayamanne commented Nov 28, 2017

In order to get the linter to run when changes are made, setting python.linting.lintOnTextChange also requires the user to turn on the setting files.autoSave. This defeats the purpose of this setting. Hence we have decided to deprecate this.

If you require linting of Python files on text change then enable the following settings:

"python.linting.lintOnSave": true,
"files.autoSave": true

If the above solution is not acceptable and you prefer the linter to run without having to save your changes, then please add your vote 👍 against the following issue #408.

@microsoft microsoft locked and limited conversation to collaborators Nov 28, 2017
@DonJayamanne DonJayamanne added the feature-request Request for new features or functionality label Nov 28, 2017
@brettcannon brettcannon modified the milestone: December 2017 Nov 28, 2017
DonJayamanne added a commit that referenced this issue Nov 28, 2017
…315)

Fixes #313 
The setting linting.lintOnTextChange is deprecated as it was never implemented.
@brettcannon brettcannon added this to the December 2017 milestone Nov 29, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-linting feature-request Request for new features or functionality
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants