Repository navigation
Ci/automerge develop to main - #85
Closed
brendanobra wants to merge 7 commits into
Closed
brendanobra wants to merge 7 commits into
brendanobra wants to merge 7 commits into
Conversation
added 2 commits
June 29, 2026 16:15
…nto ci/automerge-develop-to-main
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
There was a problem hiding this comment.
CodeQL found more than 20 potential problems in the proposed changes. Check the Files changed tab for more details.
Contributor
There was a problem hiding this comment.
Pull request overview
Adds GitHub automation to keep main synchronized with develop by introducing a composite “sync-branches” action and a consumer workflow that runs on pushes to develop (with a manual workflow_dispatch option).
Changes:
- Added a composite action (
.github/actions/sync-branches) that mergessource_branch→target_branch, and falls back to opening/updating a PR on conflicts or push failures. - Added a consumer workflow (
sync-develop-to-main.yml) and a matching consumer template to run the sync automatically (with concurrency + permissions). - Added installer lock metadata and a release helper script for moving the floating major tag.
Reviewed changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated 7 comments.
Show a summary per file
| File | Description |
|---|---|
| .github/workflows/sync-develop-to-main.yml | Consumer workflow to run develop→main sync on push/dispatch. |
| .github/automation-install.lock.json | Records installed automation module and managed files. |
| .github/actions/sync-branches/scripts/move-major-tag.sh | Release helper to move actions-v<major> tag to a stable release tag. |
| .github/actions/sync-branches/README.md | Usage/integration documentation for the sync action and installer flow. |
| .github/actions/sync-branches/consumer-template.yml | Template workflow consumers can copy to enable syncing. |
| .github/actions/sync-branches/action.yml | Composite action implementing merge/push with PR fallback behavior. |
|
|
||
| ### 1. Copy The Consumer Template | ||
|
|
||
| Copy `actions/sync-branches/consumer-template.yml` into your consumer repo as `.github/workflows/sync-develop-to-main.yml`. |
Comment on lines
+270
to
+272
| This integration guide is stored in `actions/sync-branches/` for easy discovery. | ||
|
|
||
| The composite action is at `actions/sync-branches/action.yml`. |
Comment on lines
+12
to
+15
| --version) | ||
| VERSION="$2" | ||
| shift 2 | ||
| ;; |
Contributor
There was a problem hiding this comment.
Copilot review overview
🟡 Changes recommended
The workflow bypasses its bundled action, suppresses downstream CI, and contains security and branch-update issues.
Review effort: Balanced
Findings: 4
Open (13)
set -u, passing--versionwithout a value (or as the last arg) will crash due to an…issues: write, but the action’s default behavior….github/prefix, but the action and docs are actually…pr_labelsdefault in the README (auto-sync,needs-review) does not match the composite action…actions/sync-branches/consumer-template.yml, but in this repo the template…ifbody, which makes the bash script…EXISTING=...assignment is indented deeper than the surrounding bash block, which makes the…
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Contributor
There was a problem hiding this comment.
Copilot review overview
🟡 Changes recommended
The workflow bypasses its vendored action, and repeated conflicts can leave fallback PRs stale.
Review effort: Balanced
Findings: 2
Open (9)
set -u, passing--versionwithout a value (or as the last arg) will crash due to an….github/prefix, but the action and docs are actually…actions/sync-branches/consumer-template.yml, but in this repo the template…
Resolved since last review (7)
issues: write, but the action’s default behavior…pr_labelsdefault in the README (auto-sync,needs-review) does not match the composite action…ifbody, which makes the bash script…EXISTING=...assignment is indented deeper than the surrounding bash block, which makes the…
Comment on lines
+274
to
+279
| if [[ -n "$existing_head" ]]; then | ||
| echo "Reusing existing fallback branch ${existing_head}." | ||
| echo "fallback_branch=${existing_head}" >> "$GITHUB_OUTPUT" | ||
| echo "source_sha=${source_sha}" >> "$GITHUB_OUTPUT" | ||
| echo "reason=${reason}" >> "$GITHUB_OUTPUT" | ||
| exit 0 |
Contributor
There was a problem hiding this comment.
Copilot review overview
🟡 Changes recommended
Repeated conflicts leave fallback PR branches stale, and several reliability and documentation issues remain.
Review effort: Balanced
Findings: 1
Open (10)
set -u, passing--versionwithout a value (or as the last arg) will crash due to an….github/prefix, but the action and docs are actually…actions/sync-branches/consumer-template.yml, but in this repo the template…
Resolved since last review (1)
Contributor
There was a problem hiding this comment.
Copilot review overview
🟡 Changes recommended
Conflict outcomes are misclassified, and repeated conflicts can leave fallback artifacts stale.
Review effort: Balanced
Findings: 1
Open (7)
set -u, passing--versionwithout a value (or as the last arg) will crash due to an….github/prefix, but the action and docs are actually…actions/sync-branches/consumer-template.yml, but in this repo the template…
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 subscribe to this conversation on GitHub.
Already have an account?
Sign in.
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.
No description provided.