Skip to content

Conversation

@stefanvanburen
Copy link
Member

We bumped to Go 1.23 support in #14, so we don't need to run our workflow on an unsupported version. Just run based on the go version in go.mod.

We bumped to Go 1.23 support in
#14, so we don't need to
run our workflow on an unsupported version. Just run based on the
go version in go.mod.
@stefanvanburen stefanvanburen requested review from a user and bufdev August 8, 2025 18:27
@ghost
Copy link

ghost commented Aug 13, 2025

Shouldn't we move it to test Go 1.24? (and Go 1.25, I suppose.)

@stefanvanburen stefanvanburen changed the title Remove Go 1.22 workflow in CI Bump tested Go versions in CI Aug 14, 2025
@stefanvanburen
Copy link
Member Author

Shouldn't we move it to test Go 1.24? (and Go 1.25, I suppose.)

Done. I guess we could also bump the go.mod to 1.24 here now that 1.25 is out following our support policy, but can also just leave it.

I'm also a little confused as to why the CI job isn't running, I guess because I'm editing it in this PR?

@ghost
Copy link

ghost commented Aug 29, 2025

I have no clue why the workflow isn't running. I guess it would probably work, but we should maybe figure it out.

Otherwise, we have churn in CI due to the generated files changing.
@stefanvanburen
Copy link
Member Author

I have no clue why the workflow isn't running. I guess it would probably work, but we should maybe figure it out.

Just ran it locally against this branch and it failed for an unrelated reason: https://github.com/bufbuild/bufplugin-go/actions/runs/17328800947. Pushed a fix in b6673e7, and looks like the matrix is running properly in that commit. So going to merge this if those pass.

@ghost
Copy link

ghost commented Aug 29, 2025

Yeah, @DMarby sniped the issue: the workflow got disabled since it hadn't run for 60 days. I re-enabled it. Thanks David!

@stefanvanburen
Copy link
Member Author

Yeah, @DMarby sniped the issue: the workflow got disabled since it hadn't run for 60 days. I re-enabled it. Thanks David!

huh! did not even know that's a thing. thank you both for fixing that!

@stefanvanburen stefanvanburen merged commit 7872400 into main Aug 29, 2025
5 checks passed
@stefanvanburen stefanvanburen deleted the svanburen/nix-go1.22-workflow branch August 29, 2025 16:26
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