Install binary to a less permissive location by default

egladman commented 6 months ago

I ran into a few issues when trying to run this action on self-hosted runners.

The following changes were made and are backwards compatible:

gracefully stop containers
- Previously docker stop would still be executed even if there weren't any containers present
install the dagger binary to a less permissive path
- previously the action would fail to run since /usr/local wasn't writable by default
- installs the binary to a temporary location that's accessible to the entire job/steps, then add that location to the GITHUB_PATH.

egladman commented 6 months ago

This PR goes about it in a different way, however it addresses https://github.com/dagger/dagger-for-github/issues/77

jpadams commented 6 months ago

@egladman Thank you for the PR! Reviewing it now!

dagger / dagger-for-github