The Wayback Machine - https://web.archive.org/web/20201114121838/https://github.com/letsencrypt/boulder/pull/4097
Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Go 1.12 to Travis. #4097

Merged
merged 3 commits into from Mar 6, 2019
Merged

Add Go 1.12 to Travis. #4097

merged 3 commits into from Mar 6, 2019

Conversation

@jsha
Copy link
Contributor

@jsha jsha commented Mar 6, 2019

No description provided.

@jsha jsha requested a review from letsencrypt/boulder-developers as a code owner Mar 6, 2019
@@ -111,11 +111,6 @@ fi
if [[ "$RUN" =~ "fmt" ]] ; then
start_context "fmt"
check_gofmt() {
# NOTE(@cpu): Go 1.10.3's gofmt and Go 1.11.1's gofmt don't agree. Let's

This comment has been minimized.

@cpu

cpu Mar 6, 2019
Contributor

🎉

This comment has been minimized.

@jsha

jsha Mar 6, 2019
Author Contributor

😭

This comment has been minimized.

@cpu

cpu Mar 6, 2019
Contributor

Oh noooo. I cursed it 😱

@@ -2,7 +2,7 @@ version: '3'
services:
boulder:
# To minimize fetching this should be the same version used below
image: letsencrypt/boulder-tools-go${TRAVIS_GO_VERSION:-1.11.5}:2019-02-13
image: letsencrypt/boulder-tools-go${TRAVIS_GO_VERSION:-1.12}:2019-02-13

This comment has been minimized.

@cpu

cpu Mar 6, 2019
Contributor

I think you need a s/2019-02-13/2019-03-06/g throughout docker-compose.yml

This comment has been minimized.

@cpu

cpu Mar 6, 2019
Contributor

Yeah, this is failing in CI with:

manifest for letsencrypt/boulder-tools-go1.12:2019-02-13 not found

@jsha
Copy link
Contributor Author

@jsha jsha commented Mar 6, 2019

Still failing Travis - taking a closer look now.

@cpu
cpu approved these changes Mar 6, 2019
Copy link
Contributor

@cpu cpu left a comment

Thanks @jsha!

@cpu
Copy link
Contributor

@cpu cpu commented Mar 6, 2019

Test-only: merging with one 🔍

@cpu cpu merged commit 2f6626a into master Mar 6, 2019
1 check passed
1 check passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@cpu cpu deleted the go-twelve branch Mar 6, 2019
@jsha jsha mentioned this pull request Mar 8, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.