Skip to content

Fix Github Actions Workflow#81

Merged
abhinavdangeti merged 3 commits intomasterfrom
workflow
Jan 6, 2026
Merged

Fix Github Actions Workflow#81
abhinavdangeti merged 3 commits intomasterfrom
workflow

Conversation

@CascadingRadium
Copy link
Member

  • Upgrade actions/setup-go@v1 to actions/setup-go@v5
  • Upgrade to go 1.24.0

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR attempts to modernize the GitHub Actions workflow by upgrading from actions/setup-go@v1 to v5 and updating the Go version requirements. However, the changes introduce critical issues related to Go version formats and unreleased Go versions.

  • Upgrades GitHub Actions setup-go action from v1 to v5
  • Updates Go version in go.mod from 1.21 to 1.24.0
  • Modifies the test matrix to use Go versions 1.23.x, 1.24.x, and 1.25.x

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
go.mod Updates the Go version directive, but uses incorrect format (1.24.0 instead of 1.24) and references an unreleased version
.github/workflows/tests.yml Upgrades actions/setup-go to v5 and updates test matrix with Go versions, but includes unreleased versions (1.24.x, 1.25.x)

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

abhinavdangeti
abhinavdangeti previously approved these changes Dec 30, 2025
go.mod Outdated
module github.com/blevesearch/bleve_index_api

go 1.21
go 1.24.0
Copy link
Member

Choose a reason for hiding this comment

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

Drop the .0.

@abhinavdangeti abhinavdangeti dismissed their stale review December 30, 2025 16:33

In-line comment.

@abhinavdangeti abhinavdangeti merged commit 8cdb2a0 into master Jan 6, 2026
9 checks passed
@abhinavdangeti abhinavdangeti deleted the workflow branch January 6, 2026 17:47
@github-project-automation github-project-automation bot moved this from Todo to Done in Geo-Spatial Search v2 Jan 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

2 participants