Skip to content

Suggest changes instead of automatically fixing them#26

Closed
lucic71 wants to merge 5 commits intoCpp2Rust:masterfrom
lucic71:suggestions
Closed

Suggest changes instead of automatically fixing them#26
lucic71 wants to merge 5 commits intoCpp2Rust:masterfrom
lucic71:suggestions

Conversation

@lucic71
Copy link
Copy Markdown
Contributor

@lucic71 lucic71 commented Apr 22, 2026

No description provided.

lucic71 added 5 commits April 22, 2026 17:51
format.yml, which runs on pull_request, does not have write access to
neither base nor fork repos.

To fix this, split auto-fixing in 2 steps: first format.yml publishes an
artifact with the diff, then format-apply.yml fetches the artifact and
commits the auto-fixes. This works because format-applt.yml runs on
workflow_run, not pull_request, and has write permissions.
Comment on lines +30 to +37
- name: Checkout PR head
uses: actions/checkout@v6
with:
repository: ${{ steps.head.outputs.repo }}
ref: ${{ steps.head.outputs.branch }}
token: ${{ secrets.GITHUB_TOKEN }}

- name: Apply diff
@nunoplopes nunoplopes closed this Apr 23, 2026
@lucic71 lucic71 deleted the suggestions branch April 29, 2026 13:18
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.

3 participants