Skip to content
This repository has been archived by the owner on Oct 24, 2023. It is now read-only.

chore: Update go-dev tools image for go 1.11.4 #205

Merged
merged 5 commits into from
Dec 26, 2018

Conversation

mboersma
Copy link
Member

What this PR does / why we need it:

See https://golang.org/doc/devel/release.html#go1.11.minor
and https://github.com/deis/docker-go-dev/releases/tag/v1.18.0

Release note:

chore: Update go-dev tools image for go 1.11.4

@acs-bot acs-bot added size/S and removed size/XS labels Dec 26, 2018
@mboersma mboersma changed the title chore: Update go-dev tools image for go 1.11.4 chore: Update go-dev tools and run linters in container Dec 26, 2018
@mboersma mboersma requested a review from tariq1890 December 26, 2018 19:10
tariq1890
tariq1890 previously approved these changes Dec 26, 2018
Copy link
Contributor

@tariq1890 tariq1890 left a comment

Choose a reason for hiding this comment

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

/lgtm

@acs-bot acs-bot added the lgtm label Dec 26, 2018
@mboersma mboersma changed the title chore: Update go-dev tools and run linters in container chore: Update go-dev tools image for go 1.11.4 Dec 26, 2018
@tariq1890
Copy link
Contributor

/lgtm

@acs-bot acs-bot added the lgtm label Dec 26, 2018
@acs-bot
Copy link

acs-bot commented Dec 26, 2018

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: mboersma, tariq1890

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@mboersma mboersma merged commit 60e813e into Azure:master Dec 26, 2018
@mboersma mboersma deleted the update-go-dev branch December 26, 2018 23:23
@codecov
Copy link

codecov bot commented Dec 26, 2018

Codecov Report

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

@@            Coverage Diff             @@
##           master     #205      +/-   ##
==========================================
- Coverage   53.21%   53.19%   -0.03%     
==========================================
  Files          95       95              
  Lines       14230    14230              
==========================================
- Hits         7573     7569       -4     
- Misses       5991     5995       +4     
  Partials      666      666

3 similar comments
@codecov
Copy link

codecov bot commented Dec 26, 2018

Codecov Report

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

@@            Coverage Diff             @@
##           master     #205      +/-   ##
==========================================
- Coverage   53.21%   53.19%   -0.03%     
==========================================
  Files          95       95              
  Lines       14230    14230              
==========================================
- Hits         7573     7569       -4     
- Misses       5991     5995       +4     
  Partials      666      666

@codecov
Copy link

codecov bot commented Dec 26, 2018

Codecov Report

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

@@            Coverage Diff             @@
##           master     #205      +/-   ##
==========================================
- Coverage   53.21%   53.19%   -0.03%     
==========================================
  Files          95       95              
  Lines       14230    14230              
==========================================
- Hits         7573     7569       -4     
- Misses       5991     5995       +4     
  Partials      666      666

@codecov
Copy link

codecov bot commented Dec 26, 2018

Codecov Report

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

@@            Coverage Diff             @@
##           master     #205      +/-   ##
==========================================
- Coverage   53.21%   53.19%   -0.03%     
==========================================
  Files          95       95              
  Lines       14230    14230              
==========================================
- Hits         7573     7569       -4     
- Misses       5991     5995       +4     
  Partials      666      666

@CecileRobertMichon CecileRobertMichon added this to the v0.29.0 milestone Jan 2, 2019
juhacket pushed a commit to juhacket/aks-engine that referenced this pull request Mar 14, 2019
* chore: Update go-dev tools image for go 1.11.4

See https://golang.org/doc/devel/release.html#go1.11.minor
and https://github.com/deis/docker-go-dev/releases/tag/v1.18.0

* Bump to go-dev:v1.18.1

See https://github.com/deis/docker-go-dev/releases/tag/v1.18.1

* Run linters inside go-dev container

* Increase linter timeout to 3 minutes

* Remove `make test-style` docker command for CircleCI environment
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants