Skip to content

chore: Update Go to 1.24.6 - #1147

Merged
alexeagle merged 16 commits into
mainfrom
fmeum-patch-1
Aug 15, 2025
Merged

chore: Update Go to 1.24.6#1147
alexeagle merged 16 commits into
mainfrom
fmeum-patch-1

Conversation

@fmeum

@fmeum fmeum commented Aug 15, 2025

Copy link
Copy Markdown
Member

Also enable macOS and Windows tests for all PRs.

Work towards #1146

@fmeum fmeum changed the title Update to Go 1.24.6 chore: Update to Go 1.24.6 Aug 15, 2025
@fmeum fmeum changed the title chore: Update to Go 1.24.6 chore: Update Go to 1.24.6 Aug 15, 2025
@fmeum

fmeum commented Aug 15, 2025

Copy link
Copy Markdown
Member Author

@peakschris Do you understand why Windows jobs aren't running for this PR?

@peakschris

Copy link
Copy Markdown
Contributor

Curious! Maybe this line needs relaxing?

if [[ "${{ github.ref_name }}" == "main" ]] || [[ "$HEAD_REF" == *"windows"* ]]; then

Chris

@fmeum

fmeum commented Aug 15, 2025

Copy link
Copy Markdown
Member Author

Good catch, just saw this as well and decided to drop it. Doesn't seem relevant anymore with this being part of bazel-contrib.

I'll look into cutting a release after this update.

@peakschris

Copy link
Copy Markdown
Contributor

Wonderful! love the extra windows test coverage by default 👌

@fmeum
fmeum marked this pull request as ready for review August 15, 2025 14:40
@fmeum
fmeum requested review from alexeagle and gregmagolan August 15, 2025 14:40
Comment thread .github/workflows/ci.yaml
@fmeum
fmeum enabled auto-merge (squash) August 15, 2025 15:12
@alexeagle
alexeagle disabled auto-merge August 15, 2025 15:52
@alexeagle
alexeagle merged commit fafce60 into main Aug 15, 2025
24 of 76 checks passed
@alexeagle
alexeagle deleted the fmeum-patch-1 branch August 15, 2025 15:52
alexeagle pushed a commit that referenced this pull request Aug 15, 2025
Also enable macOS and Windows tests for all PRs.

Work towards #1146
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