Skip to content

typo fix oci/typo_spec_opts_test#2653

Merged
estesp merged 2 commits intocontainerd:masterfrom
liyongxin:master
Sep 20, 2018
Merged

typo fix oci/typo_spec_opts_test#2653
estesp merged 2 commits intocontainerd:masterfrom
liyongxin:master

Conversation

@liyongxin
Copy link
Copy Markdown
Contributor

No description provided.

estesp
estesp previously approved these changes Sep 17, 2018
Copy link
Copy Markdown
Member

@estesp estesp left a comment

Choose a reason for hiding this comment

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

LGTM - but you need to sign your commit before we can merge (git commit --amend -s)

@liyongxin
Copy link
Copy Markdown
Contributor Author

@estesp tks, already signed~

Comment thread oci/spec_opts_test.go Outdated

if len(s.Process.Env) != 2 {
t.Fatal("coudn't unset")
t.Fatal("could't unset")
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

should be couldn't

@estesp estesp dismissed their stale review September 19, 2018 13:54

waiting for correction on the typo fix

Signed-off-by: Yongxin Li <[email protected]>
@liyongxin
Copy link
Copy Markdown
Contributor Author

@crosbymichael tks, have been recommited~

@codecov-io
Copy link
Copy Markdown

codecov-io commented Sep 20, 2018

Codecov Report

Merging #2653 into master will decrease coverage by 2.47%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2653      +/-   ##
==========================================
- Coverage   47.52%   45.04%   -2.48%     
==========================================
  Files          87       92       +5     
  Lines        8211    10106    +1895     
==========================================
+ Hits         3902     4552     +650     
- Misses       3591     4834    +1243     
- Partials      718      720       +2
Flag Coverage Δ
#linux 48.78% <ø> (+1.26%) ⬆️
#windows 41.8% <ø> (?)
Impacted Files Coverage Δ
snapshots/native/native.go 43.3% <0%> (-10%) ⬇️
metadata/snapshot.go 45.8% <0%> (-8.96%) ⬇️
archive/tar.go 43.13% <0%> (-6.87%) ⬇️
metadata/containers.go 47.97% <0%> (-6.62%) ⬇️
content/local/writer.go 56.25% <0%> (-6.42%) ⬇️
archive/tar_opts.go 28.57% <0%> (-4.77%) ⬇️
metadata/images.go 58.46% <0%> (-4.7%) ⬇️
metadata/buckets.go 56.33% <0%> (-4.6%) ⬇️
metadata/adaptors.go 41.46% <0%> (-4.49%) ⬇️
metadata/gc.go 61.35% <0%> (-4.22%) ⬇️
... and 54 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 ef39bba...03b1dae. Read the comment docs.

@crosbymichael
Copy link
Copy Markdown
Member

LGTM

Copy link
Copy Markdown
Member

@estesp estesp left a comment

Choose a reason for hiding this comment

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

LGTM

@estesp estesp merged commit 16b42fc into containerd:master Sep 20, 2018
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