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

Use sys.executable instead of python in Pylint plugin #327

Merged
merged 1 commit into from
Jan 1, 2023

Conversation

bnavigator
Copy link
Contributor

@bnavigator bnavigator commented Jan 1, 2023

Similar to #111, the plugin should not assume that there is a python command in the PATH. (See also PEP394)

@ccordoba12 ccordoba12 added this to the v1.7.1 milestone Jan 1, 2023
@ccordoba12 ccordoba12 changed the title Use sys.executable instead of python Use sys.executable instead of python in Pylint plugin Jan 1, 2023
Copy link
Member

@ccordoba12 ccordoba12 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me, thanks @bnavigator!

@ccordoba12 ccordoba12 merged commit 9d33ab0 into python-lsp:develop Jan 1, 2023
bmwiedemann pushed a commit to bmwiedemann/openSUSE that referenced this pull request Jan 3, 2023
https://build.opensuse.org/request/show/1045989
by user bnavigator + dimstar_suse
- Update to 1.7.0
  * Add a new plugin to provide autoimport functionality (disabled
    by default).
  * Add progress reporting.
  * Make jedi_definition plugin follow definitions to pyi files.
  * Add support for flake8 version 6.
  * Add support for Yapf ignore patterns.
  * Add mccabe setting to flake8 plugin.
- Drop python-lsp-server-pr316-flake8v6.patch upstreamed
  * gh#python-lsp/python-lsp-server#316
- Add python-lsp-server-pr327-sys-executable.patch
  * gh#python-lsp/python-lsp-server#327 (forwarded request 1045987 from bnavigator)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants