Skip to content

[29.x backport] cli/command: add missing "go:build" comments#6884

Merged
thaJeztah merged 1 commit intodocker:29.xfrom
thaJeztah:29.x_backport_missing_buildtags
Mar 25, 2026
Merged

[29.x backport] cli/command: add missing "go:build" comments#6884
thaJeztah merged 1 commit intodocker:29.xfrom
thaJeztah:29.x_backport_missing_buildtags

Conversation

@thaJeztah
Copy link
Member

cli/command: add missing "go:build" comments

  • commit e8dc2fc modernized loops to range over int, which requires go1.22 or later.
  • commit 85ebca5 modernized code to use stdlib min/max, which requires go1.21 or later.

- Human readable description for the release notes

Go SDK: Add missing build-tag, which could cause `cannot range over 10 (untyped int constant)`  when importing the `cli/command` package.

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

- commit e8dc2fc modernized loops to
  range over int, which requires go1.22 or later.
- commit 85ebca5 modernized code to
  use stdlib min/max, which requires go1.21 or later.

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
(cherry picked from commit 9a47118)
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@thaJeztah thaJeztah added impact/changelog status/2-code-review kind/bugfix PR's that fix bugs area/go-sdk Changes affecting the Go SDK labels Mar 25, 2026
@thaJeztah thaJeztah added this to the 29.3.1 milestone Mar 25, 2026
@codecov-commenter
Copy link

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@thaJeztah thaJeztah merged commit 6b3ca8f into docker:29.x Mar 25, 2026
115 of 117 checks passed
@thaJeztah thaJeztah deleted the 29.x_backport_missing_buildtags branch March 25, 2026 14:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/go-sdk Changes affecting the Go SDK impact/changelog kind/bugfix PR's that fix bugs status/2-code-review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants