ci: auto-apply rustfmt and commit formatting fixes#29
Merged
Conversation
Replace the formatting check with cargo fmt --all and use the shared commit-changes composite action to push any fixes back to the branch. Add contents: write permission so the token can push commits. Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
|
Review changes with |
|
Note Gemini is unable to generate a summary for this pull request due to the file types involved not being currently supported. |
There was a problem hiding this comment.
Pull request overview
Updates the GitHub Actions CI workflow to automatically apply rustfmt fixes and push them back to the branch, instead of failing the build on formatting differences.
Changes:
- Replace
cargo fmt -- --checkwithcargo fmt --allto auto-apply formatting. - Add a step using the shared
commit-changescomposite action to commit/push formatting updates. - Grant
contents: writepermissions so the workflow token can push commits.
Skip the commit-changes step and downgrade contents permission to read on fork PRs where GITHUB_TOKEN cannot push. Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
GitHub Actions does not support expressions in the permissions block. Revert to static contents: write — the if: condition on the commit step is sufficient to prevent pushes on fork PRs. Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
tcrypt25519
added a commit
that referenced
this pull request
Feb 16, 2026
* ci: auto-apply rustfmt and commit formatting fixes Replace the formatting check with cargo fmt --all and use the shared commit-changes composite action to push any fixes back to the branch. Add contents: write permission so the token can push commits. Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com> * fix * ci: gate rustfmt commit step to trusted contexts Skip the commit-changes step and downgrade contents permission to read on fork PRs where GITHUB_TOKEN cannot push. Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com> * ci: fix permissions expression (must be static) GitHub Actions does not support expressions in the permissions block. Revert to static contents: write — the if: condition on the commit step is sufficient to prevent pushes on fork PRs. Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com> --------- Co-authored-by: Claude Sonnet 4.5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Replace the formatting check with cargo fmt --all and use the shared commit-changes composite action to push any fixes back to the branch. Add contents: write permission so the token can push commits.