Skip to content

Conversation

@thaJeztah
Copy link
Member

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]>
@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 Aug 20, 2025
@thaJeztah thaJeztah added this to the 29.0.0 milestone Aug 20, 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!

@thaJeztah thaJeztah marked this pull request as ready for review August 20, 2025 14:08
@thaJeztah thaJeztah merged commit b0d1d94 into docker:master Aug 21, 2025
104 of 105 checks passed
@thaJeztah thaJeztah deleted the internalize_ParseEnvFile branch August 21, 2025 11:35
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 process/cherry-picked status/2-code-review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants