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

build: update CI to use Go 1.13 #3698

Merged
merged 4 commits into from
Oct 10, 2019
Merged

build: update CI to use Go 1.13 #3698

merged 4 commits into from
Oct 10, 2019

Conversation

bboreham
Copy link
Collaborator

@bboreham bboreham commented Oct 1, 2019

After #3692 we need to update the CI image. And this in turn needs a few updates in the tools subtree.

@bboreham bboreham force-pushed the update-backend-build branch 2 times, most recently from faf3ead to 567a090 Compare October 1, 2019 17:03
f041a74ff Undo some quoting that broke the test script (#160)
b1c21a068 Merge pull request #158 from weaveworks/go-1-13
d5c7dd0cd Run shell-lint during CI, and fix warnings
6db1abd14 Update to Go 1.13.1
d6cc704a2 Fix comment
7139116ae Revert "Push comments to the left so they don't appear in scripts"
e47e58f7b Push comments to the left so they don't appear in scripts
3945fcec8 Remove nonexistent env var GIT_TAG
cd6299284 Merge pull request #156 from weaveworks/drop-quay
af0eb5119 Merge pull request #157 from weaveworks/fix-image-tag-prefix-length
0b9aee4f2 Fix image-tag object name prefix length to 8 chars.
813c28fe7 Move from CircleCI 1.0 to 2.0
425cf4ef1 Move from quay.io to Dockerhub
87ccf4fd1 Merge pull request #155 from weaveworks/go-1-12
c31bc2865 Update lint script to work with Go 1.12
ed8e380d7 Update to Go 1.12.1

git-subtree-dir: tools
git-subtree-split: f041a74ffbf273b627d6c960f17357108d0dbd1c
…ackend-build

Update 'tools' subtree to latest.
It's now run in the CI for the upstream repo weaveworks/build-tools.
@bboreham bboreham force-pushed the update-backend-build branch from ec49ffe to 9e62dac Compare October 2, 2019 14:35
Copy link
Contributor

@fbarl fbarl left a comment

Choose a reason for hiding this comment

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

Thanks for this PR, LGTM! 👍

@bboreham bboreham merged commit e9e05b2 into master Oct 10, 2019
@bboreham bboreham deleted the update-backend-build branch October 10, 2019 11:32
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