-
Notifications
You must be signed in to change notification settings - Fork 12
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
Chore: update go version #53
Conversation
github.com/patrickmn/go-cache v2.1.0+incompatible | ||
github.com/stretchr/testify v1.7.0 | ||
) | ||
|
||
require ( | ||
github.com/apache/arrow/go/arrow v0.0.0-20211112161151-bc219186db40 // indirect |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we need to list these indirect dependencies? My understanding is that after installing them into go.sum you can remove them from go.mod.
@@ -1,12 +1,68 @@ | |||
module github.com/grafana/grafana-iot-twinmaker-app | |||
|
|||
go 1.16 | |||
go 1.17 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Any reason this isn't 1.18?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This signifies the minimum version that a user would need to run this package, so either 1.17 or 1.18 is fine, unless there is preference for either.
|
We no longer need this PR as we are now releasing with go 1.18 |
try building with go 1.18
Related to issue #49