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
language/python: allow python@3.7 to be used for virtualenvs #7959
Conversation
cannot be guessed automatically because a recognised Python dependency could not be found. | ||
|
||
If you are using a non-standard Python depedency, please add `:using => "python@x.y"` to | ||
`virtualenv_install_with_resources` to resolve the issue manually. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
❤️
Is it worth changing this to accept any python dependency without hardcoding an allowlist? i.e. check for any It gives us one less thing to change when |
Yes, it is definitely worth doing that. I might opt to do that after the brew release today (I don't want to set up too many road blocks) unless you all prefer for me to do it now. |
👍 to holding off for now so we can land this PR and get a release out ASAP. |
👍 totally agree |
brew style
with your changes locally?brew tests
with your changes locally?