Skip to content

deprecate ioutils.AtomicWriteFile, and use github.com/moby/sys - #3284

Draft
thaJeztah wants to merge 1 commit into
moby:masterfrom
thaJeztah:moby_atomicwriter
Draft

deprecate ioutils.AtomicWriteFile, and use github.com/moby/sys#3284
thaJeztah wants to merge 1 commit into
moby:masterfrom
thaJeztah:moby_atomicwriter

Conversation

@thaJeztah

Copy link
Copy Markdown
Member

This package was temporary until docker/pkg/ioutils was moved to a separate repository, which is now the case.

- What I did

- How I did it

- How to test it

- Description for the changelog

This package was temporary until docker/pkg/ioutils was moved to
a separate repository, which is now the case.

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
Comment thread go.mod
github.com/inconshreveable/mousetrap v1.1.0 // indirect
github.com/jmoiron/sqlx v1.3.3 // indirect
github.com/klauspost/compress v1.17.9 // indirect
github.com/moby/sys/sequential v0.7.0 // indirect

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This dependency can be dropped once we stop supporting < go1.26

@codecov-commenter

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 44.44444% with 5 lines in your changes missing coverage. Please review.
✅ Project coverage is 14.75%. Comparing base (6e9e7b8) to head (c921cee).
⚠️ Report is 60 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #3284      +/-   ##
==========================================
+ Coverage   14.73%   14.75%   +0.02%     
==========================================
  Files         200      200              
  Lines       93077    93008      -69     
==========================================
+ Hits        13712    13721       +9     
+ Misses      78019    77961      -58     
+ Partials     1346     1326      -20     
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants