Skip to content

fix(ci): retain release git credentials - #154

Merged
robjtede merged 1 commit into
mainfrom
fix/release-git-auth
Aug 6, 2026
Merged

fix(ci): retain release git credentials#154
robjtede merged 1 commit into
mainfrom
fix/release-git-auth

Conversation

@robjtede

@robjtede robjtede commented Aug 6, 2026

Copy link
Copy Markdown
Member

Keeps checkout credentials available for the release job so x52-bump-changelogs can check out and push the release PR.

The job already has scoped contents: write and pull-requests: write permissions.

Validation: zizmor v1.29.0 offline scan with no findings.

Summary by CodeRabbit

  • Chores
    • Updated the release process to retain checkout credentials during workflow execution.

@coderabbitai

coderabbitai Bot commented Aug 6, 2026

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

The release workflow now preserves checkout credentials after the checkout step.

Changes

Release workflow

Layer / File(s) Summary
Preserve checkout credentials
.github/workflows/release.yml
The checkout step changes persist-credentials from false to true.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Possibly related PRs

  • x52dev/contracts#146: This PR changes checkout credential persistence in ci.yml in the opposite direction.
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly identifies the CI change that retains Git credentials for release operations.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/release-git-auth

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

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

@robjtede
robjtede force-pushed the fix/release-git-auth branch from a111c61 to a381561 Compare August 6, 2026 13:05
@robjtede robjtede changed the title fix(ci): configure release git authentication fix(ci): retain release git credentials Aug 6, 2026
@robjtede
robjtede merged commit 4c5efe9 into main Aug 6, 2026
10 checks passed
@robjtede
robjtede deleted the fix/release-git-auth branch August 6, 2026 13:13
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