The Wayback Machine - https://web.archive.org/web/20251014182042/https://github.com/moby/moby/pull/49748
Skip to content

deprecate pkg/atomicwriter, migrate to github.com/moby/sys/atomicwriter #49748

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 1 commit into from
Apr 5, 2025
Merged

deprecate pkg/atomicwriter, migrate to github.com/moby/sys/atomicwriter #49748

merged 1 commit into from
Apr 5, 2025

Conversation

thaJeztah
Copy link
Member

@thaJeztah thaJeztah commented Apr 4, 2025 •
edited by vvoland
Loading

- Human readable description for the release notes

Go SDK: deprecate `pkg/atomicwriter`, which was migrated to `github.com/moby/sys/atomicwriter`

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

@thaJeztah thaJeztah added the kind/refactor PR's that refactor, or clean-up code label Apr 4, 2025
@thaJeztah thaJeztah added this to the 28.0.5 milestone Apr 4, 2025
@thaJeztah thaJeztah mentioned this pull request Apr 4, 2025
Open
79 tasks
@thaJeztah thaJeztah mentioned this pull request Apr 4, 2025
Merged
3 tasks
@thaJeztah
deprecate pkg/atomicwriter, migrate to github.com/moby/sys/atomicwriter
6422ff2 
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@thaJeztah thaJeztah force-pushed the migrate_atomicwriter branch from dce660f to 6422ff2 Compare April 4, 2025 21:07
@thaJeztah thaJeztah marked this pull request as ready for review April 4, 2025 21:11
@thaJeztah thaJeztah merged commit 6cbca96 into moby:master Apr 5, 2025
151 checks passed
@thaJeztah thaJeztah deleted the migrate_atomicwriter branch April 5, 2025 09:01
@thompson-shaun thompson-shaun modified the milestones: 28.0.5, 28.1.0 Apr 7, 2025
@thaJeztah thaJeztah mentioned this pull request Jun 16, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@dmcgowan dmcgowan dmcgowan approved these changes

@cpuguy83 cpuguy83 cpuguy83 approved these changes

Assignees

No one assigned

Labels

area/go-sdk impact/changelog impact/deprecation kind/refactor PR's that refactor, or clean-up code status/2-code-review

Projects

go modules transition
Status: Done

Milestone

28.1.0

Development

Successfully merging this pull request may close these issues.

4 participants

@thaJeztah @dmcgowan @cpuguy83 @thompson-shaun