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

go.mod: update dependencies and go version #248

Merged
merged 3 commits into from
Nov 5, 2022

Conversation

thaJeztah
Copy link
Contributor

go.mod: tidy with go1.17

The minimum version that's tested against is now go1.17, so updating the go.mod to that version and format;

go mod tidy -go=1.17

go.mod: golang.org/x/tools v0.2.0 to use tagged releases

full diff: golang/tools@v0.1.0...v0.2.0

go.mod: github.com/google/go-cmp v0.5.9 to remove indirect dependency

This version removes the dependency on golang.org/x/xerrors, which was used for < go1.13 compatibility.

full diff: google/go-cmp@v0.5.5...v0.5.9

The minimum version that's tested against is now go1.17, so updating
the go.mod to that version and format;

    go mod tidy -go=1.17

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
full diff: golang/tools@v0.1.0...v0.2.0

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
This version removes the dependency on golang.org/x/xerrors, which
was used for < go1.13 compatibility.

full diff: google/go-cmp@v0.5.5...v0.5.9

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
Copy link
Member

@dnephin dnephin left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@dnephin dnephin merged commit f44676a into gotestyourself:main Nov 5, 2022
@thaJeztah thaJeztah deleted the bump_deps branch November 5, 2022 19:37
@dolmen
Copy link
Contributor

dolmen commented Jun 12, 2023

@dnephin This PR is worth a new tagged release!

@dolmen dolmen mentioned this pull request Jun 29, 2023
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.

3 participants