Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@thaJeztah
Copy link
Member

@thaJeztah thaJeztah commented Aug 27, 2025

It was a wrapper around kvfile.Load, which should be used instead.

- What I did

- How I did it

- How to verify it

- Human readable description for the release notes

Go SDK: opts: deprecate `ParseEnvFile` in favour of `kvfile.Parse`.

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

It was a wrapper around kvfile.Load, which should be used instead.

Signed-off-by: Sebastiaan van Stijn <[email protected]>
(cherry picked from commit e650803)
Signed-off-by: Sebastiaan van Stijn <[email protected]>
@thaJeztah thaJeztah added this to the 28.4.0 milestone Aug 27, 2025
@thaJeztah thaJeztah added impact/changelog 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 Aug 27, 2025
@codecov-commenter
Copy link

Codecov Report

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

Files with missing lines Patch % Lines
opts/envfile_deprecated.go 0.00% 2 Missing ⚠️

📢 Thoughts on this report? Let us know!

@austinvazquez austinvazquez merged commit 69dd67f into docker:28.x Aug 27, 2025
121 of 122 checks passed
@thaJeztah thaJeztah deleted the 28.x_backport_internalize_ParseEnvFile branch August 27, 2025 23:06
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 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