Skip to content

/ python-language-server

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

Jump to bottom

Linting is disabled by default if workspace/didChangeConfiguration does not provide python settings #916

Open
Kaali opened this issue Apr 10, 2019 · 3 comments
Open

Linting is disabled by default if workspace/didChangeConfiguration does not provide python settings #916

Kaali opened this issue Apr 10, 2019 · 3 comments
Assignees
@jakebailey
Labels
bug good first issue protocol

Comments

@Kaali
Copy link

@Kaali Kaali commented Apr 10, 2019

By providing no settings workspace/didChangeConfiguration RPC throws an exception (#915). Even if the ifnull check would work, then default settings won't be applied and linting is never enabled.
@MikhailArkhipov
Copy link
Member

@MikhailArkhipov MikhailArkhipov commented Apr 11, 2019

When you say 'linting' do you mean syntax check (ie parser diagnostics) or undefined imports, etc?
@Kaali
Copy link
Author

@Kaali Kaali commented Apr 11, 2019

undefined import etc. The ones documented here: https://github.com/Microsoft/python-language-server#linting-options-diagnostics
@MikhailArkhipov
Copy link
Member

@MikhailArkhipov MikhailArkhipov commented Apr 12, 2019

@jakebailey worked on settings and related.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jakebailey jakebailey

Labels
bug good first issue protocol
Projects
None yet
Milestone
No milestone
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants
@Kaali @jakebailey @MikhailArkhipov
You can’t perform that action at this time.