Skip to content

ci: move the responder, shellcheck and tf-lint callers to shared-workflows v1.6.0; README no longer says @main - #266

Merged
cpitzi merged 1 commit into
mainfrom
ci-shared-workflows-v1.6.0
Nov 2, 2026
Merged

cpitzi merged 1 commit into
mainfrom
ci-shared-workflows-v1.6.0

Conversation

@cpitzi

@cpitzi cpitzi commented Nov 2, 2026

Copy link
Copy Markdown
Collaborator

Closes #265. Supersedes Dependabot's #228.

What changes

Why

After #264 moved docs-check to v1.6.0, the repo's callers sat on three tags. Dependabot's #228 (v1.4.0 → v1.5.0 for four callers) now conflicts on the docs-check line. Bumping the remaining three straight to v1.6.0 resolves both; Dependabot should close #228 as up to date once this merges.

Scope and trade-offs

  • No caller inputs, secrets or permissions change.
    • v1.5.0 bumped claude-code-action, which the responder uses.
    • v1.6.0's notes say these three callers "have nothing to change" and differ from v1.5.1 only in the internal render-claude-summary ref.
  • claude-review.yml stays at @v1.5.0. A PR that changes the reviewer's own caller workflow makes claude-code-action skip, so it would get a "not a review" notice instead of a review. That one-line bump goes in its own PR with a local review.
  • No Terraform files. The terraform gate skips the plan.

🤖 Generated with Claude Code

…flows v1.6.0; README no longer says @main

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@github-actions

github-actions Bot commented Nov 2, 2026

Copy link
Copy Markdown
Contributor

Review (haiku, claude-review.yml)

No significant problems found. The diff only moves three workflow caller pins from @v1.4.0 to @v1.6.0 and rewrites one README table row. It touches no Terraform, so the apply blast radius is zero. Every caller stays tag-pinned, permissions and inputs are unchanged, and the README row now matches the files in .github/workflows. The PR body has Closes #265, the title matches the body, and the body carries a full summary. I could not confirm from this environment that the v1.6.0 tag in lentago/shared-workflows contains the reusable workflows the PR relies on, so that check is still open.

@cpitzi

cpitzi commented Nov 2, 2026

Copy link
Copy Markdown
Collaborator Author

Triage of the Haiku review: no findings. The one open check is confirmed:

  • claude-responder.yml, shellcheck.yml and tf-lint.yml all exist at lentago/shared-workflows@v1.6.0. The contents API at ref=v1.6.0 returns blob SHAs a50d776, 732385a and 3345d4f.
  • The shellcheck / shellcheck and tf-lint / tf-lint checks on this PR already ran green from the v1.6.0 callers.
  • The responder runs only on issue and PR comments, so this PR does not exercise it.

Arming.

@cpitzi
cpitzi merged commit 1059d91 into main Nov 2, 2026
11 checks passed
@cpitzi
cpitzi deleted the ci-shared-workflows-v1.6.0 branch November 2, 2026 20:15
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.

ci: shared-workflows callers sit on three different tags, and the README says docs-check runs @main

1 participant