Skip to content

Conversation

@thaJeztah
Copy link
Member


It it just a chain of New("sometag").Parse(...), and most of our uses don't use a tag for the template, so can call Parse.

There's no public users of this function, but deprecating it first just in case.

- Human readable description for the release notes

Go SDK: templates: deprecate NewParse() function.

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

It it just a chain of `New("sometag").Parse(...)`, and most of our
uses don't use a tag for the template, so can call Parse.

There's no public users of this function, but deprecating it first
just in case.

Signed-off-by: Sebastiaan van Stijn <[email protected]>
(cherry picked from commit 7ab3e7e)
Signed-off-by: Sebastiaan van Stijn <[email protected]>
@thaJeztah thaJeztah added this to the 28.4.1 milestone Sep 24, 2025
@thaJeztah thaJeztah added impact/deprecation status/2-code-review kind/refactor PR's that refactor, or clean-up code area/go-sdk Changes affecting the Go SDK impact/go-sdk Noteworthy (compatibility changes) in the Go SDK labels Sep 24, 2025
@codecov-commenter
Copy link

Codecov Report

❌ Patch coverage is 80.00000% with 1 line in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
templates/templates.go 50.00% 1 Missing ⚠️

📢 Thoughts on this report? Let us know!

@thaJeztah thaJeztah merged commit d48256b into docker:28.x Sep 24, 2025
118 of 119 checks passed
@thaJeztah thaJeztah deleted the 28.x_backport_template_deprecate_newparse branch September 24, 2025 11:44
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/deprecation impact/go-sdk Noteworthy (compatibility changes) in the Go SDK kind/refactor PR's that refactor, or clean-up code status/2-code-review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants