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 #268

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from
Draft

Conversation

thaJeztah
Copy link
Contributor

go.mod: golang.org/x/sys v0.1.0

The golang.org/x/,, projects now tag releases, so updating to the first tagged release.

go.mod: github.com/sirupsen/logrus v1.9.0

Updating to the current release, which has been out for a while, and
most projects have already updated to this version, so let's update
it here as well to reduces the number of versions in the dependency
tree.

full diff: sirupsen/logrus@v1.7.0...v1.9.0

This removes golang.org/x/tools/cmd/stringer as a direct dependency of this
module, instead using "go install" to install the required tool as part of
`go generate`. I opted for pinning to a fixed version for reproducibility, but
we could switch to `@latest` if there's no concerns for getting newer versions.

With this patch:

    which stringer
    # (no output)

    go generate ./pkg/guid
    which stringer
    /Users/thajeztah/go/bin/stringer

Signed-off-by: Sebastiaan van Stijn <[email protected]>
The golang.org/x/,, projects now tag releases, so updating to the
first tagged release.

Signed-off-by: Sebastiaan van Stijn <[email protected]>
Updating to the current release, which has been out for a while, and
most projects have already updated to this version, so let's update
it here as well to reduces the number of versions in the dependency
tree.

full diff: sirupsen/logrus@v1.7.0...v1.9.0

Signed-off-by: Sebastiaan van Stijn <[email protected]>
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.

1 participant