Skip to content

platforms: Add MustParse#2548

Merged
estesp merged 1 commit intocontainerd:masterfrom
ijc:platforms-must-parse
Aug 15, 2018
Merged

platforms: Add MustParse#2548
estesp merged 1 commit intocontainerd:masterfrom
ijc:platforms-must-parse

Conversation

@ijc
Copy link
Copy Markdown
Contributor

@ijc ijc commented Aug 15, 2018

This function is analogous to regexp.MustCompile and can simplify production
of a Platform from a hard-coded strings, e.g. for global variable
initialisation.

Signed-off-by: Ian Campbell [email protected]

This function is analogous to `regexp.MustCompile` and can simplify production
of a `Platform` from a hard-coded strings, e.g. for global variable
initialisation.

Signed-off-by: Ian Campbell <[email protected]>
@codecov-io
Copy link
Copy Markdown

Codecov Report

Merging #2548 into master will decrease coverage by 0.02%.
The diff coverage is 0%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2548      +/-   ##
==========================================
- Coverage   44.95%   44.93%   -0.03%     
==========================================
  Files          95       95              
  Lines        9928     9933       +5     
==========================================
  Hits         4463     4463              
- Misses       4743     4748       +5     
  Partials      722      722
Flag Coverage Δ
#linux 48.85% <0%> (-0.04%) ⬇️
#windows 41.54% <0%> (-0.03%) ⬇️
Impacted Files Coverage Δ
platforms/platforms.go 79.34% <0%> (-4.57%) ⬇️

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 037fc5e...c83f977. 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 a69a0b0 into containerd:master Aug 15, 2018
@ijc ijc deleted the platforms-must-parse branch August 15, 2018 14:22
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