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

deps: declare go 1.18 in go.mod #329

Merged
merged 2 commits into from
Dec 23, 2022
Merged

Conversation

whereswaldon
Copy link
Contributor

This updates the go.{mod,sum} files to declare a version of Go that supports module graph pruning. The change allows consuming projects to avoid picking up dependencies on modules that ghw only uses for testing or for its CLI.

Fixes #328

Signed-off-by: Chris Waldon christopher.waldon.dev@gmail.com

This updates the go.{mod,sum} files to declare a version of Go that supports
module graph pruning. The change allows consuming projects to avoid picking
up dependencies on modules that ghw only uses for testing or for its CLI.

Fixes jaypipes#328

Signed-off-by: Chris Waldon <christopher.waldon.dev@gmail.com>
@ffromani
Copy link
Collaborator

thanks for this PR! I'd be happy to approve considering @jaypipes seems already happy with this.
Could you please check if our github actions configuration needs updates? It should be simple as doing version bumps.
It would be nice to have go.{mod,sum} and CI always aligned

Signed-off-by: Chris Waldon <christopher.waldon.dev@gmail.com>
@whereswaldon
Copy link
Contributor Author

I've attempted to update the Go versions for CI.

@ffromani
Copy link
Collaborator

ffromani commented Dec 23, 2022

Thanks a lot @whereswaldon , looks great! I'm gonna wait a little time to let @jaypipes chime in (e.g. to assess how this change interacts [if at all] with future releases plans, because we're jumping quite a bit of versions), if not I'll merge later considering the past conversation in the issue. Thanks again!

Copy link
Owner

@jaypipes jaypipes left a comment

Choose a reason for hiding this comment

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

Excellent, thank you so much @whereswaldon! :)

@jaypipes jaypipes merged commit 77ab5ed into jaypipes:main Dec 23, 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.

Consider updating declared Go version in go.mod
3 participants