Skip to content

ci: simplify translate workflow by using gh instead of peter-evans/create-pull-request#2237

Merged
dunglas merged 3 commits intomainfrom
ci/simplify-translate
Mar 9, 2026
Merged

ci: simplify translate workflow by using gh instead of peter-evans/create-pull-request#2237
dunglas merged 3 commits intomainfrom
ci/simplify-translate

Conversation

@dunglas
Copy link
Member

@dunglas dunglas commented Mar 5, 2026

No description provided.

Copilot AI review requested due to automatic review settings March 5, 2026 17:04
@dunglas dunglas changed the title ci: simplify translate workflow by using gh instead of peter-evans/cr… ci: simplify translate workflow by using gh instead of peter-evans/create-pull-request Mar 5, 2026
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.


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

Comment on lines 20 to 22
uses: actions/checkout@v6
with:
fetch-depth: 0
Copy link

Copilot AI Mar 6, 2026

Choose a reason for hiding this comment

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

Removing persist-credentials: false deviates from the convention used in every other workflow in this repo (see lint.yaml:26, tests.yaml:42, docker.yaml:58, static.yaml:69, windows.yaml:63, etc.). While this removal is necessary because git push needs credentials, consider adding a comment explaining why persist-credentials: false is intentionally omitted here, so future reviewers don't add it back and break the workflow.

Copilot uses AI. Check for mistakes.
@dunglas dunglas force-pushed the ci/simplify-translate branch from 846c64a to 628d2e0 Compare March 9, 2026 13:04
@dunglas dunglas merged commit 16a70e4 into main Mar 9, 2026
31 checks passed
@dunglas dunglas deleted the ci/simplify-translate branch March 9, 2026 14:55
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