Skip to content

feat: Do not raise multiple sync PRs#175

Merged
nurof3n merged 1 commit into
prod-stagingfrom
danielvallance/single_sync_pr
May 28, 2026
Merged

feat: Do not raise multiple sync PRs#175
nurof3n merged 1 commit into
prod-stagingfrom
danielvallance/single_sync_pr

Conversation

@danielvallance

Copy link
Copy Markdown
Contributor

This changes the sync workflow to push changes
to a single, fixed, branch name. This means that
there will only ever be a single open sync PR.

Closes: FIELD-510

This changes the sync workflow to push changes
to a single, fixed, branch name. This means that
there will only ever be a single open sync PR.

Closes: FIELD-510

Signed-off-by: Daniel Vallance <daniel@unikraft.com>

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the scheduled/manual sync workflow so that guide updates are always pushed to a single fixed branch, ensuring there is only one open “sync” PR at a time (FIELD-510).

Changes:

  • Remove per-run “Generate branch ID” logic that created a unique branch name on each execution.
  • Configure peter-evans/create-pull-request to always use the fixed branch sync/examples.

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

@nurof3n nurof3n left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM!

Reviewed-by: Alex-Andrei Cioc andrei.cioc@unikraft.io
Approved-by: Alex-Andrei Cioc andrei.cioc@unikraft.io

@nurof3n nurof3n merged commit 1030c10 into prod-staging May 28, 2026
10 checks passed
@nurof3n nurof3n deleted the danielvallance/single_sync_pr branch May 28, 2026 09:25
@nurof3n nurof3n temporarily deployed to pull-requests May 28, 2026 09:25 — with GitHub Actions Inactive
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