Skip to content

[27.x backport] pkg/ioutils: deprecate NopWriter, NopWriteClose#49255

Merged
thaJeztah merged 2 commits intomoby:27.xfrom
thaJeztah:27.x_backport_ioutils_more_deprecations
Jan 10, 2025
Merged

[27.x backport] pkg/ioutils: deprecate NopWriter, NopWriteClose#49255
thaJeztah merged 2 commits intomoby:27.xfrom
thaJeztah:27.x_backport_ioutils_more_deprecations

Conversation

@thaJeztah
Copy link
Copy Markdown
Member

- Description for the changelog

Go SDK: pkg/ioutils: deprecate `NopWriter` in favour of `io.Discard`. It will be removed in the next release.
Go SDK: pkg/ioutils: deprecate `NopWriteCloser`. It was only used internally, and will be removed in the next release.

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

It's not used, and users can use io.Discard instead.

Signed-off-by: Sebastiaan van Stijn <[email protected]>
(cherry picked from commit 7fa3c55)
Signed-off-by: Sebastiaan van Stijn <[email protected]>
It was only used internally, and has no external consumers; deprecate
it to be removed in the next release.

Signed-off-by: Sebastiaan van Stijn <[email protected]>
(cherry picked from commit 3faa170)
Signed-off-by: Sebastiaan van Stijn <[email protected]>
@thaJeztah thaJeztah added this to the 27.5.0 milestone Jan 10, 2025
@thaJeztah thaJeztah self-assigned this Jan 10, 2025
@thaJeztah
Copy link
Copy Markdown
Member Author

Failure is unrelated; known flaky;

=== FAIL: amd64.integration-cli TestDockerSwarmSuite/TestSwarmLockUnlockCluster (68.73s)

@thaJeztah thaJeztah merged commit 38b84dc into moby:27.x Jan 10, 2025
@thaJeztah thaJeztah deleted the 27.x_backport_ioutils_more_deprecations branch January 10, 2025 14:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants