Skip to content

fix(ci): pin auto-rebase reusable workflow to SHA#153

Closed
don-petry wants to merge 12 commits into
mainfrom
claude/issue-144-20260501-1332
Closed

fix(ci): pin auto-rebase reusable workflow to SHA#153
don-petry wants to merge 12 commits into
mainfrom
claude/issue-144-20260501-1332

Conversation

@don-petry

@don-petry don-petry commented May 1, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Pins petry-projects/.github/.github/workflows/auto-rebase-reusable.yml from @v1 to its resolved commit SHA 126c1441ee9cf040f2ce3ef0eda85d459b82f8e9
  • Retains # v1 comment so the human-readable version is still visible
  • Brings the repository into compliance with the org-wide action-pinning policy

Closes #144

Generated with Claude Code

Summary by CodeRabbit

  • Chores
    • Updated GitHub Actions workflow configuration to pin a reusable workflow dependency to a specific commit hash instead of a version reference, ensuring consistent workflow execution.

Pin petry-projects/.github auto-rebase-reusable.yml from @v1 to SHA
126c1441ee9cf040f2ce3ef0eda85d459b82f8e9 to comply with the
org-wide action-pinning policy.

Closes #144

Co-authored-by: don-petry <don-petry@users.noreply.github.com>
Copilot AI review requested due to automatic review settings May 1, 2026 13:34
@coderabbitai

coderabbitai Bot commented May 1, 2026

Copy link
Copy Markdown

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 16b1ac5a-91d5-4625-a3b2-ea3eb332c86c

📥 Commits

Reviewing files that changed from the base of the PR and between 2ecfdb3 and 4c23597.

📒 Files selected for processing (1)
  • .github/workflows/auto-rebase.yml

📝 Walkthrough

Walkthrough

Updates .github/workflows/auto-rebase.yml to pin the reusable workflow reference to a specific commit SHA instead of a version tag, addressing action-pinning compliance requirements.

Changes

Cohort / File(s) Summary
Workflow Action Pinning
.github/workflows/auto-rebase.yml
Pins reusable workflow reference from @v1 tag to commit SHA @126c1441ee9cf040f2ce3ef0eda85d459b82f8e9, retaining the version comment for reference clarity.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related issues

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: pinning the auto-rebase reusable workflow from @v1 to a fixed commit SHA.
Linked Issues check ✅ Passed The PR successfully addresses issue #144 by pinning the auto-rebase reusable workflow to commit SHA 126c1441ee9cf040f2ce3ef0eda85d459b82f8e9, bringing the repository into compliance with the org-wide action-pinning policy.
Out of Scope Changes check ✅ Passed All changes are scoped to the objective: pinning the reusable workflow reference and retaining the human-readable comment, with only a trailing newline adjustment.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch claude/issue-144-20260501-1332

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share
Review rate limit: 0/1 reviews remaining, refill in 60 minutes.

Comment @coderabbitai help to get the list of available commands and usage tips.

@don-petry

Copy link
Copy Markdown
Contributor Author

Self-review complete — the change looks correct:

  • SHA 126c1441ee9cf040f2ce3ef0eda85d459b82f8e9 is the resolved commit for the v1 tag (verified via GitHub API)
  • Only the uses: line was modified; the trigger event, concurrency group, and job-level permissions: block are all unchanged (as required by the file's AGENTS header)
  • # v1 comment retained for human readability

Waiting for CI. @don-petry — CODEOWNERS designates you as reviewer. Please review and merge when checks pass.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Pins the org-level reusable workflow reference in the repo’s auto-rebase caller workflow to a specific commit SHA to satisfy the action-pinning compliance requirement.

Changes:

  • Update .github/workflows/auto-rebase.yml to use a commit SHA instead of @v1 for petry-projects/.github/.github/workflows/auto-rebase-reusable.yml.
  • Preserve the # v1 inline comment for human-readable version context.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@github-actions github-actions Bot requested a review from a team as a code owner May 4, 2026 20:38
@don-petry don-petry closed this May 11, 2026
@don-petry don-petry reopened this May 11, 2026
@sonarqubecloud

Copy link
Copy Markdown

@don-petry don-petry enabled auto-merge (squash) May 12, 2026 01:41
@donpetry-bot

Copy link
Copy Markdown
Contributor

@donpetry-bot I'm on it — starting a fresh review now. Results will appear in a few minutes.

5 similar comments
@donpetry-bot

Copy link
Copy Markdown
Contributor

@donpetry-bot I'm on it — starting a fresh review now. Results will appear in a few minutes.

@donpetry-bot

Copy link
Copy Markdown
Contributor

@donpetry-bot I'm on it — starting a fresh review now. Results will appear in a few minutes.

@donpetry-bot

Copy link
Copy Markdown
Contributor

@donpetry-bot I'm on it — starting a fresh review now. Results will appear in a few minutes.

@donpetry-bot

Copy link
Copy Markdown
Contributor

@donpetry-bot I'm on it — starting a fresh review now. Results will appear in a few minutes.

@donpetry-bot

Copy link
Copy Markdown
Contributor

@donpetry-bot I'm on it — starting a fresh review now. Results will appear in a few minutes.

@donpetry-bot

Copy link
Copy Markdown
Contributor

Automated review — human attention needed

This PR has been through 3 automated review cycles (cap: 3) without converging on an approval-and-merge state. Further automated review has been paused to avoid infinite loops.

Please take a look manually, or close this PR if it's no longer needed. Once a human review resolves the situation, remove the needs-human-review label and the cascade can be re-engaged on the next push.

Posted by the donpetry-bot PR-review cascade.

@donpetry-bot

Copy link
Copy Markdown
Contributor

@donpetry-bot I'm on it — starting a fresh review now. Results will appear in a few minutes.

6 similar comments
@donpetry-bot

Copy link
Copy Markdown
Contributor

@donpetry-bot I'm on it — starting a fresh review now. Results will appear in a few minutes.

@donpetry-bot

Copy link
Copy Markdown
Contributor

@donpetry-bot I'm on it — starting a fresh review now. Results will appear in a few minutes.

@donpetry-bot

Copy link
Copy Markdown
Contributor

@donpetry-bot I'm on it — starting a fresh review now. Results will appear in a few minutes.

@donpetry-bot

Copy link
Copy Markdown
Contributor

@donpetry-bot I'm on it — starting a fresh review now. Results will appear in a few minutes.

@donpetry-bot

Copy link
Copy Markdown
Contributor

@donpetry-bot I'm on it — starting a fresh review now. Results will appear in a few minutes.

@donpetry-bot

Copy link
Copy Markdown
Contributor

@donpetry-bot I'm on it — starting a fresh review now. Results will appear in a few minutes.

@donpetry-bot

Copy link
Copy Markdown
Contributor

Automated review — human attention needed

This PR has been through 3 automated review cycles (cap: 3) without converging on an approval-and-merge state. Further automated review has been paused to avoid infinite loops.

Please take a look manually, or close this PR if it's no longer needed. Once a human review resolves the situation, remove the needs-human-review label and the cascade can be re-engaged on the next push.

Posted by the donpetry-bot PR-review cascade.

@donpetry-bot

Copy link
Copy Markdown
Contributor

@donpetry-bot I'm on it — starting a fresh review now. Results will appear in a few minutes.

7 similar comments
@donpetry-bot

Copy link
Copy Markdown
Contributor

@donpetry-bot I'm on it — starting a fresh review now. Results will appear in a few minutes.

@donpetry-bot

Copy link
Copy Markdown
Contributor

@donpetry-bot I'm on it — starting a fresh review now. Results will appear in a few minutes.

@donpetry-bot

Copy link
Copy Markdown
Contributor

@donpetry-bot I'm on it — starting a fresh review now. Results will appear in a few minutes.

@donpetry-bot

Copy link
Copy Markdown
Contributor

@donpetry-bot I'm on it — starting a fresh review now. Results will appear in a few minutes.

@donpetry-bot

Copy link
Copy Markdown
Contributor

@donpetry-bot I'm on it — starting a fresh review now. Results will appear in a few minutes.

@donpetry-bot

Copy link
Copy Markdown
Contributor

@donpetry-bot I'm on it — starting a fresh review now. Results will appear in a few minutes.

@donpetry-bot

Copy link
Copy Markdown
Contributor

@donpetry-bot I'm on it — starting a fresh review now. Results will appear in a few minutes.

@donpetry-bot

Copy link
Copy Markdown
Contributor

Automated review — human attention needed

This PR has been through 3 automated review cycles (cap: 3) without converging on an approval-and-merge state. Further automated review has been paused to avoid infinite loops.

Please take a look manually, or close this PR if it's no longer needed. Once a human review resolves the situation, remove the needs-human-review label and the cascade can be re-engaged on the next push.

Posted by the donpetry-bot PR-review cascade.

@donpetry-bot

Copy link
Copy Markdown
Contributor

@donpetry-bot I'm on it — starting a fresh review now. Results will appear in a few minutes.

5 similar comments
@donpetry-bot

Copy link
Copy Markdown
Contributor

@donpetry-bot I'm on it — starting a fresh review now. Results will appear in a few minutes.

@donpetry-bot

Copy link
Copy Markdown
Contributor

@donpetry-bot I'm on it — starting a fresh review now. Results will appear in a few minutes.

@donpetry-bot

Copy link
Copy Markdown
Contributor

@donpetry-bot I'm on it — starting a fresh review now. Results will appear in a few minutes.

@donpetry-bot

Copy link
Copy Markdown
Contributor

@donpetry-bot I'm on it — starting a fresh review now. Results will appear in a few minutes.

@donpetry-bot

Copy link
Copy Markdown
Contributor

@donpetry-bot I'm on it — starting a fresh review now. Results will appear in a few minutes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs-human-review Requires human review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Compliance: unpinned-actions-auto-rebase.yml

3 participants