Skip to content

scripts/sync-branch: Fix non-interactive merge#7098

Merged
thaJeztah merged 1 commit into
docker:masterfrom
vvoland:sync-editor
Jul 10, 2026
Merged

scripts/sync-branch: Fix non-interactive merge#7098
thaJeztah merged 1 commit into
docker:masterfrom
vvoland:sync-editor

Conversation

@vvoland

@vvoland vvoland commented Jul 9, 2026

Copy link
Copy Markdown
Collaborator

- What I did

- How I did it

- How to verify it

- Human readable description for the release notes

- A picture of a cute animal (not mandatory but encouraged)

@vvoland vvoland added this to the 29.7.0 milestone Jul 9, 2026

@docker-agent docker-agent 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.

Assessment: 🟢 APPROVE

The change is surgical and correct: adding --no-edit to both git merge invocations in scripts/sync-branch prevents Git from opening an interactive editor to compose the merge-commit message. This is the proper fix for non-interactive (CI) environments. No bugs, logic errors, or regressions were identified.

@codecov-commenter

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

Signed-off-by: Paweł Gronowski <pawel.gronowski@docker.com>

@thaJeztah thaJeztah left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@thaJeztah thaJeztah merged commit a8ddac1 into docker:master Jul 10, 2026
95 checks passed
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.

4 participants