Skip to content

ale_python_pyls_options #3033

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

Closed
n8henrie opened this issue Mar 8, 2020 · 1 comment · Fixed by #3620
Closed

ale_python_pyls_options #3033

n8henrie opened this issue Mar 8, 2020 · 1 comment · Fixed by #3620

Comments

@n8henrie
Copy link
Contributor

n8henrie commented Mar 8, 2020

pyls is conspicuously missing from the list of python linters that supports additional command line arguments:

$ curl -s https://raw.githubusercontent.com/dense-analysis/ale/master/doc/ale-python.txt | awk '$1 ~ /^g:.*_options/ { print $1 }'
g:ale_python_autopep8_options
g:ale_python_bandit_options
g:ale_python_black_options
g:ale_python_flake8_options
g:ale_python_isort_options
g:ale_python_mypy_options
g:ale_python_prospector_options
g:ale_python_pycodestyle_options
g:ale_python_pydocstyle_options
g:ale_python_pylama_options
g:ale_python_pylint_options
g:ale_python_reorder_python_imports_options
g:ale_python_vulture_options

It's been discussed in the context of changing the pyls configuration, but my use case (and I suspect the more common use case) would be for invoking pyls as a module, like this example in the documentation for flake8 .

For some reason my python install loses its pyls executable frequently, even though the module remains and is still works fine with python -m pyls. I've tried let g:ale_python_pyls_executable = "/usr/local/bin/python3 -m pyls" but this doesn't work. Being able to follow the example of several other python linters and use:

let g:ale_python_pyls_executable = 'python3'
let g:ale_python_pyls_options = '-m pyls'

seems like it would do the trick. I'd be happy to work on a PR if this seems acceptable.

@w0rp
Copy link
Member

w0rp commented Aug 29, 2020

Yeah, add a pull request for it. Just make it clear what the difference between _config and _options is, in the documentation.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants