Skip to content

[ca] Minor cleanups#2067

Merged
cyli merged 1 commit into
moby:masterfrom
cyli:minor-cleanup
Mar 29, 2017
Merged

[ca] Minor cleanups#2067
cyli merged 1 commit into
moby:masterfrom
cyli:minor-cleanup

Conversation

@cyli
Copy link
Copy Markdown
Contributor

@cyli cyli commented Mar 29, 2017

While re-organizing #2052, pulled out some minor changes that don't really fit. This just adds an additional test case I had forgotten for validating intermediates, and cleans up an extra variable declaration.

…mediate

must have the same subject and public key as the signing cert), and cleaning
up an extra useless variable.

Signed-off-by: cyli <ying.li@docker.com>
@aaronlehmann
Copy link
Copy Markdown
Collaborator

LGTM

@codecov
Copy link
Copy Markdown

codecov Bot commented Mar 29, 2017

Codecov Report

Merging #2067 into master will increase coverage by 0.1%.
The diff coverage is 100%.

@@           Coverage Diff            @@
##           master   #2067     +/-   ##
========================================
+ Coverage    54.2%   54.3%   +0.1%     
========================================
  Files         111     111             
  Lines       19352   19351      -1     
========================================
+ Hits        10489   10508     +19     
+ Misses       7595    7582     -13     
+ Partials     1268    1261      -7

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 b4c4309...71c5111. Read the comment docs.

@cyli
Copy link
Copy Markdown
Contributor Author

cyli commented Mar 29, 2017

Just going to merge this one, because it's super tiny.

@cyli cyli merged commit d4d2e26 into moby:master Mar 29, 2017
@cyli cyli deleted the minor-cleanup branch March 29, 2017 00:57
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