Skip to content

bump docker/go-units v0.4.0#3237

Merged
crosbymichael merged 1 commit intocontainerd:masterfrom
thaJeztah:bump_go_units
Apr 23, 2019
Merged

bump docker/go-units v0.4.0#3237
crosbymichael merged 1 commit intocontainerd:masterfrom
thaJeztah:bump_go_units

Conversation

@thaJeztah
Copy link
Copy Markdown
Member

relevant changes:

- docker/go-units#19 make 1 second not to be plural seconds
- docker/go-units#20 Add `HumanSizeWithPrecision` function
- docker/go-units#21 change week display rule
- docker/go-units#22 Better human duration precision
- docker/go-units#23 Removes spaces before unit
- docker/go-units#27 Fix containerd#26 - RAMInBytes Bug
- docker/go-units#33 Fix handling of unlimited (-1) ulimit values
- docker/go-units#34 Revert 46 minute threshold

Signed-off-by: Sebastiaan van Stijn <[email protected]>
@thaJeztah
Copy link
Copy Markdown
Member Author

@AkihiroSuda

Copy link
Copy Markdown
Member

@AkihiroSuda AkihiroSuda left a comment

Choose a reason for hiding this comment

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

LGTM if green

@codecov-io
Copy link
Copy Markdown

codecov-io commented Apr 23, 2019

Codecov Report

Merging #3237 into master will increase coverage by 4.02%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #3237      +/-   ##
==========================================
+ Coverage   44.63%   48.65%   +4.02%     
==========================================
  Files         113      102      -11     
  Lines       12161     9569    -2592     
==========================================
- Hits         5428     4656     -772     
+ Misses       5898     4087    -1811     
+ Partials      835      826       -9
Flag Coverage Δ
#linux 48.65% <ø> (ø) ⬆️
#windows ?
Impacted Files Coverage Δ
remotes/docker/auth.go 63.82% <0%> (-3.97%) ⬇️
remotes/docker/status.go 21.42% <0%> (-3.58%) ⬇️
remotes/docker/fetcher.go 61.53% <0%> (-3.05%) ⬇️
platforms/cpuinfo.go 3.77% <0%> (-0.85%) ⬇️
log/context.go 36.84% <0%> (-0.66%) ⬇️
namespaces/context.go 55% <0%> (-0.56%) ⬇️
errdefs/grpc.go 74.6% <0%> (-0.4%) ⬇️
remotes/docker/pusher.go 0% <0%> (ø) ⬆️
content/local/locks.go 100% <0%> (ø) ⬆️
labels/validate.go 100% <0%> (ø) ⬆️
... and 63 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 835e6d0...c79c792. Read the comment docs.

@crosbymichael
Copy link
Copy Markdown
Member

LGTM

@crosbymichael crosbymichael merged commit 2d780a7 into containerd:master Apr 23, 2019
@thaJeztah thaJeztah deleted the bump_go_units branch April 23, 2019 21:28
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.

4 participants