Skip to content

pkg/compose: go fix#13782

Merged
ndeloof merged 1 commit into
docker:mainfrom
thaJeztah:go_fix
May 11, 2026
Merged

pkg/compose: go fix#13782
ndeloof merged 1 commit into
docker:mainfrom
thaJeztah:go_fix

Conversation

@thaJeztah
Copy link
Copy Markdown
Member

What I did

Related issue

(not mandatory) A picture of a cute animal, if possible in relation to what you did

Signed-off-by: Sebastiaan van Stijn <[email protected]>
Copilot AI review requested due to automatic review settings May 7, 2026 22:32
@thaJeztah thaJeztah requested a review from a team as a code owner May 7, 2026 22:32
@thaJeztah thaJeztah requested review from glours and ndeloof May 7, 2026 22:32
Copy link
Copy Markdown
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

Updates a small string helper in the Compose plugin execution path to use a more idiomatic standard-library API for extracting the first line of a message.

Changes:

  • Replace manual newline search (strings.IndexByte) with strings.Cut in firstLine.

Copy link
Copy Markdown

@docker-agent docker-agent Bot left a comment

Choose a reason for hiding this comment

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

Assessment: 🟢 APPROVE

This PR refactors in to use instead of + manual slicing. The two approaches are semantically equivalent — no bugs introduced.

Copy link
Copy Markdown

@docker-agent docker-agent Bot left a comment

Choose a reason for hiding this comment

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

Assessment: 🟢 APPROVE

This PR refactors firstLine in pkg/compose/plugins.go to use strings.Cut instead of strings.IndexByte + manual slicing. The two approaches are semantically equivalent — no bugs introduced.

@codecov
Copy link
Copy Markdown

codecov Bot commented May 7, 2026

Codecov Report

❌ Patch coverage is 0% with 2 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
pkg/compose/plugins.go 0.00% 1 Missing and 1 partial ⚠️

📢 Thoughts on this report? Let us know!

@thaJeztah
Copy link
Copy Markdown
Member Author

🎉 Got 3 bots (one of which TWICE) that like my PR 😂

@ndeloof ndeloof merged commit 8854550 into docker:main May 11, 2026
46 checks passed
@thaJeztah thaJeztah deleted the go_fix branch May 11, 2026 14:31
tmeijn pushed a commit to tmeijn/dotfiles that referenced this pull request May 21, 2026
This MR contains the following updates:

| Package | Update | Change |
|---|---|---|
| [docker/compose](https://github.com/docker/compose) | patch | `v5.1.3` → `v5.1.4` |

MR created with the help of [el-capitano/tools/renovate-bot](https://gitlab.com/el-capitano/tools/renovate-bot).

**Proposed changes to behavior should be submitted there as MRs.**

---

### Release Notes

<details>
<summary>docker/compose (docker/compose)</summary>

### [`v5.1.4`](https://github.com/docker/compose/releases/tag/v5.1.4)

[Compare Source](docker/compose@v5.1.3...v5.1.4)

#### What's Changed

##### ✨ Improvements

- feat: add stop lifecycle hook for external providers by [@&#8203;glours](https://github.com/glours) in [#&#8203;13779](docker/compose#13779)

##### 🐛 Fixes

- fix: route OCI artifact pulls through Docker Desktop HTTP proxy by [@&#8203;glours](https://github.com/glours) in [#&#8203;13770](docker/compose#13770)
- fix: restore stoppingEvent/stoppedEvent helpers for plugin stop hook by [@&#8203;glours](https://github.com/glours) in [#&#8203;13794](docker/compose#13794)
- fix(publish): flag literal inline environment values by [@&#8203;glours](https://github.com/glours) in [#&#8203;13760](docker/compose#13760)

##### 🔧  Internal

- ci: remove unused e2e job from merge workflow by [@&#8203;glours](https://github.com/glours) in [#&#8203;13740](docker/compose#13740)
- chore: update cagent-action to `v1.4.4` by [@&#8203;derekmisler](https://github.com/derekmisler) in [#&#8203;13745](docker/compose#13745)
- Change verb tense in Docker Compose reference documentation by [@&#8203;ryanjbonnell](https://github.com/ryanjbonnell) in [#&#8203;13773](docker/compose#13773)
- pkg/compose: go fix by [@&#8203;thaJeztah](https://github.com/thaJeztah) in [#&#8203;13782](docker/compose#13782)
- refactor: code deduplication and simplification by [@&#8203;ndeloof](https://github.com/ndeloof) in [#&#8203;13759](docker/compose#13759)
- fix: make e2e tests pass reliably locally with Docker Desktop by [@&#8203;glours](https://github.com/glours) in [#&#8203;13741](docker/compose#13741)
- refactor: drop Desktop beta-settings check; gate hint on LogsTab flag by [@&#8203;glours](https://github.com/glours) in [#&#8203;13755](docker/compose#13755)

##### ⚙️ Dependencies

- build(deps): bump github.com/mattn/go-shellwords from `1.0.12` to `1.0.13` by [@&#8203;dependabot](https://github.com/dependabot)\[bot] in [#&#8203;13731](docker/compose#13731)
- build(deps): bump github.com/docker/cli from `29.4.0+incompatible` to `29.4.2+incompatible` by [@&#8203;dependabot](https://github.com/dependabot)\[bot] in [#&#8203;13768](docker/compose#13768)
- build(deps): bump github.com/moby/moby/client from `0.4.0` to `0.4.1` by [@&#8203;dependabot](https://github.com/dependabot)\[bot] in [#&#8203;13752](docker/compose#13752)
- build(deps): bump github.com/docker/cli from `29.4.2+incompatible` to `29.4.3+incompatible` by [@&#8203;dependabot](https://github.com/dependabot)\[bot] in [#&#8203;13776](docker/compose#13776)
- build(deps): bump google.golang.org/grpc from `1.80.0` to `1.81.0` by [@&#8203;dependabot](https://github.com/dependabot)\[bot] in [#&#8203;13775](docker/compose#13775)
- build(deps):  update to `go 1.26.3` by [@&#8203;thaJeztah](https://github.com/thaJeztah) in [#&#8203;13783](docker/compose#13783)
- build(deps): bump google.golang.org/grpc from `1.81.0` to `1.81.1` by [@&#8203;dependabot](https://github.com/dependabot)\[bot] in [#&#8203;13791](docker/compose#13791)
- build(deps): bump github.com/compose-spec/compose-go/v2 from `2.10.2` to `2.11.0` by [@&#8203;dependabot](https://github.com/dependabot)\[bot] in [#&#8203;13798](docker/compose#13798)
- build(deps): bump github.com/docker/cli from `29.4.3+incompatible` to `29.5.1+incompatible` by [@&#8203;dependabot](https://github.com/dependabot)\[bot] in [#&#8203;13796](docker/compose#13796)
- build(deps): bump golang.org/x/sys from `0.42.0` to `0.44.0` by [@&#8203;dependabot](https://github.com/dependabot)\[bot] in [#&#8203;13788](docker/compose#13788)

#### New Contributors

- [@&#8203;ryanjbonnell](https://github.com/ryanjbonnell) made their first contribution in [#&#8203;13773](docker/compose#13773)

**Full Changelog**: <docker/compose@v5.1.3...v5.1.4>

</details>

---

### Configuration

📅 **Schedule**: (UTC)

- Branch creation
  - At any time (no schedule defined)
- Automerge
  - At any time (no schedule defined)

🚦 **Automerge**: Enabled.

♻ **Rebasing**: Whenever MR is behind base branch, or you tick the rebase/retry checkbox.

🔕 **Ignore**: Close this MR and you won't be reminded about this update again.

---

 - [ ] <!-- rebase-check -->If you want to rebase/retry this MR, check this box

---

This MR has been generated by [Mend Renovate](https://github.com/renovatebot/renovate).
<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiI0My4xODYuNCIsInVwZGF0ZWRJblZlciI6IjQzLjE4Ni40IiwidGFyZ2V0QnJhbmNoIjoibWFpbiIsImxhYmVscyI6WyJSZW5vdmF0ZSBCb3QiLCJhdXRvbWF0aW9uOmJvdC1hdXRob3JlZCIsImRlcGVuZGVuY3ktdHlwZTo6cGF0Y2giXX0=-->
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