Use pyenv to find executable only if pyenv is installed and a local shim exists for the cwd #839
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
gamboz
force-pushed
the
feature/issue-126__pyenv-no-local
branch
from
c05a5ce to
76ffaf5
Compare
gamboz
force-pushed
the
feature/issue-126__pyenv-no-local
branch
from
76ffaf5 to
f96ae9f
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
I have a problem similar to that discussed in #126 i.e. I have
pyenvinstalled, but I'm working onpyvenvenvironment with nopyenv"local" application-specific Python version for the current project / cwd.Here is a "this-works-for-me" change that lets dap-mode use
pyenvonly ifpyenv localreturns something different than "pyenv: no local version configured for this directory"Unfortunately I'm not expert enough to be sure that what I'm proposing here is always a good solution, so please feel very free to close this PR 🙂