-
Notifications
You must be signed in to change notification settings - Fork 925
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
Document the minimum dependencies necessary to use this action with custom containers #399
Comments
@kevinburkemeter thanks. We're going to fallback to downloading the repo using the REST API if git isnt in the PATH. |
will be fixed by pr actions/checkout#104 |
@kevinburkemeter we automatically inject node into the container as that is a runtime concern you should not have to deal with. |
I'm still seeing this with actions/checkout@v2 and also
|
Transferring to actions/runner This looks related to minimum dependencies to run any javascript action. Related to #334 ? |
is still seeing the issue: |
Hi @igagis! Is this issue still a problem on a newer runner version? |
@nikola-jokic I don't know, I'm not using official runner for self-hosted machines anymore, I'm using https://github.com/ChristopherHX/github-act-runner and also I don't use javascript-based actions in my pipelines anymore. So, I cannot check, sorry. |
Since this issue is related to actions/checkout, but we use this issue board to track issues with the runner, I am closing this issue. |
It would be nice to know the minimum amount of software necessary to run this with a custom container. For example, this project requires
git
at at least v2.18.I'm currently trying to debug this failure:
I suspect this means I need to add Node.js to the container image but I'm not sure.
The text was updated successfully, but these errors were encountered: