Skip to content
This repository has been archived by the owner on May 18, 2021. It is now read-only.

feat: build with go 1.13 #210

Merged
merged 1 commit into from
Sep 5, 2019
Merged

feat: build with go 1.13 #210

merged 1 commit into from
Sep 5, 2019

Conversation

nickatsegment
Copy link
Contributor

@nickatsegment nickatsegment commented Sep 5, 2019

Needed to bump xerrors to fix their go1.13 support.

Should cut a release after. The xerrors bump should fix Rowno's problem with our aws-okta homebrew recipe failing to build.

Fixes #209

@nickatsegment nickatsegment force-pushed the go1.13 branch 6 times, most recently from 093ec11 to 440a5a0 Compare September 5, 2019 18:58
@nickatsegment nickatsegment changed the title go 1.13 feat: go 1.13 Sep 5, 2019
@nickatsegment nickatsegment changed the title feat: go 1.13 feat: build with go 1.13 Sep 5, 2019
@nickatsegment nickatsegment marked this pull request as ready for review September 5, 2019 19:01
Copy link
Member

@Fauzyy Fauzyy left a comment

Choose a reason for hiding this comment

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

👍

@nickatsegment nickatsegment merged commit bedb81c into master Sep 5, 2019
@nickatsegment nickatsegment deleted the go1.13 branch September 5, 2019 20:13
@@ -1,8 +1,25 @@
version: 2
jobs:
test:

test-build-golang-prev:
docker:
- image: circleci/golang:1.12
Copy link

Choose a reason for hiding this comment

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

Shouldn't you run the tests on golang:1.13 too? 🤔

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I guess it couldn't hurt, but I haven't seen it be a problem yet, so we'll cross that bridge when we get to it :)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Use go 1.13
3 participants