Invalid comment in LSP-pylsp.sublime-settings
StabbarN opened this issue · comments
See
LSP-pylsp/LSP-pylsp.sublime-settings
Line 62 in 81fabfe
flake8
is invalid according to
https://github.com/sublimelsp/LSP-pylsp/blob/master/sublime-package.json#L35-L38
Should it be pyflakes
instead? Flake8 is a wrapper around pyflakes (and more).
The default configuration source is pycodestyle. Change the pylsp.configurationSources setting to ['flake8'] in order to respect flake8 configuration instead.
that sentence is from this README https://github.com/python-lsp/python-lsp-server#configuration
I did a quick check of the python-lsp-server repo and found this
python-lsp-server\pylsp\config\config.py
DEFAULT_CONFIG_SOURCES = ['pycodestyle']
sources = self._settings.get('configurationSources', DEFAULT_CONFIG_SOURCES)
---
python-lsp-server\test\plugins\test_flake8_lint.py
workspace.update_config({"pylsp": {"configurationSources": ["flake8"]}})
So I would say that the acceptable values are: pycodestyle
and flake8
That makes more sense. So https://github.com/sublimelsp/LSP-pylsp/blob/master/sublime-package.json#L37 should be corrected.
Yes, feel free to open a PR :)
PR was merged.