Skip to content

Bump Go to 1.8.4#2395

Merged
nishanttotla merged 1 commit intomoby:masterfrom
thaJeztah:bump-golang-to-1.8.4
Oct 5, 2017
Merged

Bump Go to 1.8.4#2395
nishanttotla merged 1 commit intomoby:masterfrom
thaJeztah:bump-golang-to-1.8.4

Conversation

@thaJeztah
Copy link
Copy Markdown
Member

Bumps the Go version used to 1.8.4, which contains security fixes; https://groups.google.com/forum/#!topic/golang-announce/1hZYiemnkdE

Bumps the Go version used to 1.8.4, which contains security fixes;
https://groups.google.com/forum/#!topic/golang-announce/1hZYiemnkdE

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@codecov-io
Copy link
Copy Markdown

codecov-io commented Oct 4, 2017

Codecov Report

Merging #2395 into master will increase coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #2395      +/-   ##
==========================================
+ Coverage   60.49%   60.49%   +<.01%     
==========================================
  Files         128      128              
  Lines       26303    26303              
==========================================
+ Hits        15912    15913       +1     
- Misses       8988     9005      +17     
+ Partials     1403     1385      -18

Copy link
Copy Markdown
Contributor

@nishanttotla nishanttotla left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Copy Markdown
Collaborator

@aaronlehmann aaronlehmann left a comment

Choose a reason for hiding this comment

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

I don't think any of the security issues are applicable, but this looks fine.

@thaJeztah
Copy link
Copy Markdown
Member Author

Yes it's only hygiene

@nishanttotla nishanttotla merged commit 78e63b3 into moby:master Oct 5, 2017
@thaJeztah thaJeztah deleted the bump-golang-to-1.8.4 branch October 5, 2017 06:58
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