Skip to content

ci: replace floating branch refs with version tags and track actions via dependabot - #319

Merged
STRRL merged 1 commit into
masterfrom
ci/tame-floating-action-refs
Jul 19, 2026
Merged

ci: replace floating branch refs with version tags and track actions via dependabot#319
STRRL merged 1 commit into
masterfrom
ci/tame-floating-action-refs

Conversation

@STRRL

@STRRL STRRL commented Jul 19, 2026

Copy link
Copy Markdown
Owner

Problem

Four workflow references point at moving branches (claude-code-action@beta x2, helm-gh-pages@master, setup-minikube@latest). Branch refs change on every upstream push, making them the largest supply chain surface in the workflows: there is no review step between upstream pushing code and our CI executing it with repository secrets. We evaluated full SHA pinning (#312) and declined it to keep receiving upstream updates; this is the middle route.

Solution

Replace branch refs with version tags, and let dependabot track all action versions weekly so updates keep arriving as reviewable PRs. Combined with the repository-level actions allowlist (configured separately in settings), workflows stay current without executing unreviewed upstream pushes.

Major Changes

  • workflow refs
    • anthropics/claude-code-action: @beta (stale branch, last updated Aug 2025) to @v1 stable; v1 renames direct_prompt to prompt, migrated accordingly
    • medyagh/setup-minikube: @latest to @v0
    • stefanprodan/helm-gh-pages: @master to @v1.7.0
  • dependabot
    • new github-actions ecosystem entry, weekly, grouped into a single PR

…via dependabot

Branch refs (beta, master, latest) move on every upstream push and are
the largest supply chain surface in the workflows. Version tags plus
weekly dependabot tracking keep updates flowing through reviewable PRs
instead. claude-code-action moves from the stale beta branch to v1,
which renames direct_prompt to prompt.
@codecov

codecov Bot commented Jul 19, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 47.58%. Comparing base (a5cc475) to head (e491f0a).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #319      +/-   ##
==========================================
- Coverage   53.26%   47.58%   -5.68%     
==========================================
  Files          12       13       +1     
  Lines         796      891      +95     
==========================================
  Hits          424      424              
- Misses        351      446      +95     
  Partials       21       21              
Flag Coverage Δ
e2e 0.00% <ø> (?)
integration 29.43% <ø> (ø)
unit 38.75% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@STRRL
STRRL marked this pull request as ready for review July 19, 2026 22:14
@cursor

cursor Bot commented Jul 19, 2026

Copy link
Copy Markdown

Bugbot is not enabled for your account, so this pull request was not reviewed.

Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs.

@STRRL
STRRL enabled auto-merge (squash) July 19, 2026 22:15
@STRRL
STRRL merged commit 6762af2 into master Jul 19, 2026
7 checks passed
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.

1 participant