Skip to content

[release/1.0] cherrypick Fix NPE in dialer#2034

Merged
stevvooe merged 1 commit intocontainerd:release/1.0from
estesp:cherrypick-npe-fix
Jan 20, 2018
Merged

[release/1.0] cherrypick Fix NPE in dialer#2034
stevvooe merged 1 commit intocontainerd:release/1.0from
estesp:cherrypick-npe-fix

Conversation

@estesp
Copy link
Copy Markdown
Member

@estesp estesp commented Jan 19, 2018

Cherry-pick #2022

Signed-off-by: Phil Estes [email protected]
Signed-off-by: Li Yi [email protected]

@estesp estesp added this to the 1.0.2 milestone Jan 19, 2018
@codecov-io
Copy link
Copy Markdown

Codecov Report

Merging #2034 into release/1.0 will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##           release/1.0    #2034   +/-   ##
============================================
  Coverage        50.59%   50.59%           
============================================
  Files               81       81           
  Lines             7165     7165           
============================================
  Hits              3625     3625           
  Misses            2847     2847           
  Partials           693      693
Flag Coverage Δ
#linux 50.59% <ø> (ø) ⬆️

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 d2179c2...a03fb1b. Read the comment docs.

@crosbymichael
Copy link
Copy Markdown
Member

LGTM

1 similar comment
@dmcgowan
Copy link
Copy Markdown
Member

LGTM

@estesp
Copy link
Copy Markdown
Member Author

estesp commented Jan 19, 2018

Not sure why Windows CI didn't run; do I need to re-push to kick it?

@stevvooe
Copy link
Copy Markdown
Member

@estesp I don't think it runs on this branch. We'll have to fix that for 1.1.

@stevvooe
Copy link
Copy Markdown
Member

LGTM

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.

6 participants