Skip to content

🔧 fix(ci): restore git credentials for release job#472

Merged
gaborbernat merged 1 commit intomainfrom
fix/release-persist-credentials
Mar 31, 2026
Merged

🔧 fix(ci): restore git credentials for release job#472
gaborbernat merged 1 commit intomainfrom
fix/release-persist-credentials

Conversation

@gaborbernat
Copy link
Copy Markdown
Member

@gaborbernat gaborbernat commented Mar 31, 2026

The zizmor security tool added persist-credentials: false to all checkout steps, which broke releases. The release job pushes version tags and changelog commits to the repository, but stripping credentials after checkout caused these operations to fail with authentication errors.

Removing persist-credentials: false from the release job's checkout restores the ability to push. 🔐 The artipacked warning is suppressed with an inline ignore because it's a false positive in this context—the release job never uploads artifacts that could leak the persisted credentials. The build job (which does upload artifacts) correctly keeps persist-credentials: false.

This change only affects the release workflow. Regular CI checks remain protected with credential stripping intact.

@gaborbernat gaborbernat requested a review from ofek as a code owner March 31, 2026 20:56
@gaborbernat gaborbernat changed the title Remove persist-credentials: false from release job 🔧 fix(ci): restore git credentials for release job Mar 31, 2026
@gaborbernat gaborbernat merged commit cb88156 into main Mar 31, 2026
32 checks passed
@gaborbernat gaborbernat deleted the fix/release-persist-credentials branch March 31, 2026 21:09
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