chore(deps): bump EarthBuild/actions-setup from cae2d9ab68894d8402751fe42e07c7cca0272f7f to 3eefe0d8a96e1ffbefb86e2bbf6542eec78cbccb#1764
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 1bc55a44c1
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
| skip_build: ${{ steps.check_images.outputs.skip_build }} | ||
| steps: | ||
| - uses: EarthBuild/actions-setup@cae2d9ab68894d8402751fe42e07c7cca0272f7f | ||
| - uses: EarthBuild/actions-setup@3eefe0d8a96e1ffbefb86e2bbf6542eec78cbccb |
There was a problem hiding this comment.
Install the
earthly binary, not earth
This new action ref no longer installs an earthly executable: the upstream source at 3eefe0d sets pkgName = 'earth' and downloads from EarthBuild/earthbuild, while the previous ref set pkgName = 'earthly' and downloaded from earthly/earthly (see https://raw.githubusercontent.com/EarthBuild/actions-setup/3eefe0d8a96e1ffbefb86e2bbf6542eec78cbccb/src/setup.ts). On a clean GitHub-hosted runner, and on any self-hosted runner without a preinstalled earthly, later steps in this workflow still invoke earthly (for example the build step), so the job fails with command not found; the same pin was updated in the other Earthly workflows as well.
Useful? React with 👍 / 👎.
Bumps [EarthBuild/actions-setup](https://github.com/earthbuild/actions-setup) from cae2d9ab68894d8402751fe42e07c7cca0272f7f to 3eefe0d8a96e1ffbefb86e2bbf6542eec78cbccb. - [Release notes](https://github.com/earthbuild/actions-setup/releases) - [Commits](EarthBuild/actions-setup@cae2d9a...3eefe0d) --- updated-dependencies: - dependency-name: EarthBuild/actions-setup dependency-version: 3eefe0d8a96e1ffbefb86e2bbf6542eec78cbccb dependency-type: direct:production ... Signed-off-by: dependabot[bot] <support@github.com>
1bc55a4 to
84c9a3d
Compare
Signed-off-by: Lech Głowiak <lech.glowiak@shielded.io>
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: a3977f84c2
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
| - uses: EarthBuild/actions-setup@cae2d9ab68894d8402751fe42e07c7cca0272f7f | ||
| with: | ||
| version: v0.8.16 | ||
| version: v0.8.17 |
There was a problem hiding this comment.
Use the EarthBuild setup pin in local env tests
This composite still uses the old setup action ref, whose setup.ts hard-codes pkgName = 'earthly' and downloads from github.com/earthly/earthly/.../${pkgName}-...; pairing it with v0.8.17 breaks the continuous-integration.yml local-environment-tests job on clean runners because that release is the EarthBuild rename (earth binary under EarthBuild/earthbuild). Even if a runner had an old toolcache entry, the later step now invokes earth, which this old setup action does not install, so this action needs the new EarthBuild setup ref as well.
Useful? React with 👍 / 👎.
Signed-off-by: Lech Głowiak <lech.glowiak@shielded.io>
a3977f8 to
651b702
Compare
Bumps EarthBuild/actions-setup from cae2d9ab68894d8402751fe42e07c7cca0272f7f to 3eefe0d8a96e1ffbefb86e2bbf6542eec78cbccb.
Commits
3eefe0dchore(deps): update alpine docker tag to v3.24 (#342)a7321ecfix(deps): update dependency semver to v7.8.5 (#339)45da143build(deps): Bump undici from 6.26.0 to 6.27.0 (#341)a31c9b8chore(deps): update alpine:3.23 docker digest to fd791d7 (#340)92db38achore(deps): update dependency@rollup/rollup-linux-x64-muslto v4.62.2 (#338)f6467f7chore(deps): lock file maintenance (#331)33a0135chore(deps): update vitest monorepo to v4.1.9 (#336)6a48230chore(deps): update typescript-eslint monorepo to v8.61.1 (#335)d9c2961fix(deps): update dependency@actions/cacheto v6.1.0 (#337)a25f63fchore(deps): update dependency@vercel/nccto v0.44.0 (#328)