Skip to content

Update sync action to overwrite develop in mirror#8690

Merged
seshubaws merged 1 commit intodevelopfrom
seshubaws-patch-1
Feb 25, 2026
Merged

Update sync action to overwrite develop in mirror#8690
seshubaws merged 1 commit intodevelopfrom
seshubaws-patch-1

Conversation

@seshubaws
Copy link
Copy Markdown
Contributor

Which issue(s) does this change fix?

--force-with-lease fails because the action does not fetch the mirror every time. We could do this, but since we want to overwrite the dev branch of the mirror repo every time, we can just use --force since no actual updates should be pushed to the dev branch of the mirror repo, only to other branches.

Why is this change necessary?

How does it address the issue?

What side effects does this change have?

Mandatory Checklist

PRs will only be reviewed after checklist is complete

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@seshubaws seshubaws requested a review from a team as a code owner February 24, 2026 22:50
@seshubaws seshubaws enabled auto-merge February 24, 2026 23:07
@reedham-aws
Copy link
Copy Markdown
Contributor

There have been merges to develop in the other repo before, would that conflict with this?

@seshubaws
Copy link
Copy Markdown
Contributor Author

There have been merges to develop in the other repo before, would that conflict with this?

Do you mean dependabot updates? That should be ok because it should be the same in both repos. There shouldn't be actual feature updates merged to develop though, that should all be done in branches, and then directly merged to public repo through PR. I've updated the release process to state this too

@seshubaws seshubaws added this pull request to the merge queue Feb 25, 2026
Merged via the queue into develop with commit b2b9f18 Feb 25, 2026
46 checks passed
@seshubaws seshubaws deleted the seshubaws-patch-1 branch March 4, 2026 18:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants