-
Notifications
You must be signed in to change notification settings - Fork 581
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
pip install
ing PyYAML crashes the Action running
#114
Comments
Experiencing the same issue here https://github.com/nathan-weinberg/jeeves/pull/133/checks?check_run_id=875661858 |
Seems like a bug for me. In the official documentation the action is used without virtualenv: https://docs.github.com/en/actions/language-and-framework-guides/using-python-with-github-actions |
I was able to workaround this by changing the pyyaml version from 5.1 to 5.3.1 in my requirements.txt |
Hello everyone, this issue is related to image, not the setup-python task. We are working on investigation and fix. Progress can be tracked in actions/runner-images#1202 (comment) |
@nathan-weinberg I have tried to force |
@Borda Here is the PR in which I made the change - https://github.com/nathan-weinberg/jeeves/pull/135/files You may need to change the action version to v2 |
Hello @jameshiebert! |
@jameshiebert I'm going to close the issue, but feel free to contact us if you have any questions. |
Describe the bug
pip install
ing a version of PyYAML which is not the version being used to run the workflow causes the workflow runner to crash on subsequent steps.Which version of the action are you using?
v1
v2
v2.0.1
ormaster
)Environment
If applicable, please specify if you're using a container
Python Versions
3.7.8
To Reproduce
Steps to reproduce the behavior:
pip installs
some version of PyYAMLRun/Repo Url
https://github.com/jameshiebert/action/runs/875331441?check_suite_focus=true
Additional context
I guess the right (?) way to work around this is to install everything in a virtual environment (as shown in the following commit), but:
If I'm doing something wrong, I'd be happy to hear about it :) Thanks.
The text was updated successfully, but these errors were encountered: