Skip to content

ci(deps): bump carabiner-dev/actions from 1.2.1 to 1.2.3 - #760

Merged
marcusburghardt merged 2 commits into
mainfrom
ci/bump-carabiner-actions-1.2.3
Jul 23, 2026
Merged

ci(deps): bump carabiner-dev/actions from 1.2.1 to 1.2.3#760
marcusburghardt merged 2 commits into
mainfrom
ci/bump-carabiner-actions-1.2.3

Conversation

@yvonnedevlinrh

@yvonnedevlinrh yvonnedevlinrh commented Jul 21, 2026

Copy link
Copy Markdown
Contributor

Bump install/snappy and install/ampel together from v1.2.1 to v1.2.3.

Why a combined PR?

These actions are co-released from the carabiner-dev/actions monorepo and share release tags. Dependabot generates individual PRs for each action path, but bumping them independently creates a version mismatch that breaks the cross-repo integration test:

Error: provider ampel (evaluator "ampel"): required tools not found: snappy

This PR supersedes #751 and #752 by updating both action pins atomically.

Changes

Action Old New
install/snappy 94f2939... (v1.2.1) 2a4b2cd... (v1.2.3)
install/ampel 94f2939... (v1.2.1) 2a4b2cd... (v1.2.3)

Both actions now set install-dir: /home/runner/.carabiner explicitly to work around an upstream bug in v1.2.3 where the default $HOME/.carabiner is written unexpanded to GITHUB_PATH, making installed tools invisible to subsequent steps. Upstream issue pending on carabiner-dev/actions.

Upstream release notes

  • v1.2.2: Ampel installer updates (v1.3.0 → v1.3.1), carabiner login action, SLSA provenance generator update
  • v1.2.3: Ampel installer update (v1.3.3), ampel-version input for verify action, internal pin workflow

Note: v1.2.3 has an upstream bug where the default install-dir ($HOME/.carabiner) is written unexpanded to GITHUB_PATH, making installed tools invisible to later steps. The second commit works around this by setting install-dir explicitly to /home/runner/.carabiner. Upstream issue pending on carabiner-dev/actions.

Closes #751, closes #752

@yvonnedevlinrh
yvonnedevlinrh requested a review from a team as a code owner July 21, 2026 13:26
@yvonnedevlinrh yvonnedevlinrh self-assigned this Jul 21, 2026
@github-actions

Copy link
Copy Markdown
Contributor

CRAP Load Analysis

No Go code changes detected in this PR. No CRAP impact.

@marcusburghardt marcusburghardt left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Bump install/snappy and install/ampel together to v1.2.3.

These actions are co-released from the carabiner-dev/actions monorepo
and must be updated atomically — bumping them individually (as
Dependabot does) creates a version mismatch that breaks the
cross-repo integration test.

Supersedes: #751, #752

Assisted-by: OpenCode (claude-opus-4-6)
Signed-off-by: Yvonne Devlin <ydevlin@redhat.com>
The v1.2.3 add-to-path step writes unexpanded $HOME to
GITHUB_PATH when using the default install-dir, making snappy
and ampel invisible to subsequent steps. Use an explicit
absolute path until the upstream fix lands.

Assisted-by: OpenCode (claude-opus-4-6)
Signed-off-by: Yvonne Devlin <ydevlin@redhat.com>

@em-redhat em-redhat left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@marcusburghardt
marcusburghardt merged commit e50f60e into main Jul 23, 2026
34 checks passed
@marcusburghardt
marcusburghardt deleted the ci/bump-carabiner-actions-1.2.3 branch July 23, 2026 13:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants