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: Bump Go version in go.mod #410

Merged
merged 1 commit into from
Feb 10, 2022

Conversation

rhcarvalho
Copy link
Contributor

@rhcarvalho rhcarvalho commented Feb 8, 2022

Commit generated with this command

go mod tidy -go=1.17 -compat=1.15

Using Go version 1.17.5.

The idea is to enable consumers of sentry-go to be able to use module
graph pruning (Go 1.17+ required), such that dependencies of sentry-go
that are otherwise unused by end users can be pruned away.

See #156 (comment)

Also update CI workflow to check for go.mod tidiness using the same line
as used to generate this commit. Update data race tests that were still
accidentally being run on Go 1.16 instead of 1.17.

@rhcarvalho
Copy link
Contributor Author

@dashed you're so fast :-)
👋

Commit generated with this command
  go mod tidy -go=1.17 -compat=1.15

Using Go version 1.17.5.

The idea is to enable consumers of sentry-go to be able to use module
graph pruning (Go 1.17+ required), such that dependencies of sentry-go
that are otherwise unused by end users can be pruned away.

See getsentry#156 (comment)

Also update CI workflow to check for go.mod tidiness using the same line
as used to generate this commit. Update data race tests that were still
accidentally being run on Go 1.16 instead of 1.17.
@rhcarvalho
Copy link
Contributor Author

The CodeQL failure can likely be ignored. It should have been addressed by github/codeql-go#658 (which closed github/codeql-go#650).

@kamilogorek do you know who's taking a look here these days?

@dashed dashed requested a review from a team February 9, 2022 02:59
@kamilogorek
Copy link
Contributor

such that dependencies of sentry-go that are otherwise unused by end users can be pruned away.

At last 😌

@kamilogorek do you know who's taking a look here these days?

Noone just yet 😔

@kamilogorek kamilogorek merged commit dc2eb25 into getsentry:master Feb 10, 2022
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