Skip to content

Update Go to 1.25 and CodeQL workflow - #413

Merged
Madhan Raj Mookkandy (madhanrm) merged 7 commits into
mainfrom
copilot/update-go-version-1-25
Jan 22, 2026
Merged

Update Go to 1.25 and CodeQL workflow#413
Madhan Raj Mookkandy (madhanrm) merged 7 commits into
mainfrom
copilot/update-go-version-1-25

Conversation

Copilot AI commented Jan 21, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Go Version Update

  • go.mod: Updated to Go 1.25.0 (toolchain directive removed, now implicit)
  • .pipelines/build.yaml: Updated GoTool@0 tasks to version 1.25.3
  • go.sum: Regenerated with Go 1.25 dependencies and committed to repository

Lint Build Fix

  • golangci-lint target: Added tidy dependency to ensure go.sum is up-to-date before linting
  • Force rebuild: Remove cached golangci-lint binary and clear Go build cache before reinstalling
  • This ensures golangci-lint is built with the correct Go 1.25 toolchain, fixing the error: "the Go language version (go1.24) used to build golangci-lint is lower than the targeted Go version (1.25.0)"

CodeQL Workflow Update

  • CodeQL Actions: Updated github/codeql-action from v2 to v3 (init, autobuild, analyze steps)
  • Checkout Action: Updated actions/checkout from v3 to v4
  • Resolves configuration compatibility warning: "Code scanning cannot determine the alerts introduced by this pull request, because 1 configuration present on refs/heads/main was not found"

Testing

  • ✅ golangci-lint runs successfully with Go 1.25
  • ✅ Unit tests pass
  • ✅ Code review completed with no issues
  • ✅ CodeQL workflow updated to latest stable versions

The go.sum file is tracked in the repository and contains the dependency checksums for Go 1.25.

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

@madhanrm
Update Go version to 1.25
988daaf 
Co-authored-by: madhanrm <20309044+madhanrm@users.noreply.github.com>
Copilot AI changed the title [WIP] Update Go version to 1.25 Update Go to 1.25 Jan 21, 2026
Copilot AI and others added 2 commits January 21, 2026 05:56
@madhanrm
Remove go.sum and generate it dynamically via make targets
d80aa2e 
Co-authored-by: madhanrm <20309044+madhanrm@users.noreply.github.com>
@madhanrm
Add tidy dependency to unittest target
a21c316 
Co-authored-by: madhanrm <20309044+madhanrm@users.noreply.github.com>
Copilot AI changed the title Update Go to 1.25 Update Go to 1.25 and generate go.sum dynamically Jan 21, 2026
@madhanrm
Madhan Raj Mookkandy (madhanrm) marked this pull request as ready for review January 21, 2026 06:03
@madhanrm

Madhan Raj Mookkandy (madhanrm) commented Jan 21, 2026

Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines

azure-pipelines Bot commented Jan 21, 2026

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@madhanrm

Madhan Raj Mookkandy (madhanrm) commented Jan 21, 2026

Copy link
Copy Markdown
Collaborator

Copilot can you fix the lint build error

@madhanrm
Fix golangci-lint build error with Go 1.25
0f46c52 
- Add tidy dependency to golangci-lint target
- Remove cached golangci-lint binary before reinstall
- Clear Go build cache to force recompilation with correct Go version

Co-authored-by: madhanrm <20309044+madhanrm@users.noreply.github.com>

Copilot AI commented Jan 21, 2026

Copy link
Copy Markdown
Contributor Author

Copilot can you fix the lint build error

Fixed in commit 0f46c52. The lint error was caused by golangci-lint being built with Go 1.24 while the project targets Go 1.25. The fix forces a clean rebuild of golangci-lint by removing the cached binary and clearing the build cache before installation.

@madhanrm

Madhan Raj Mookkandy (madhanrm) commented Jan 22, 2026

Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines

azure-pipelines Bot commented Jan 22, 2026

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@madhanrm

Madhan Raj Mookkandy (madhanrm) commented Jan 22, 2026

Copy link
Copy Markdown
Collaborator

Copilot can you retain go.sum

@madhanrm

Madhan Raj Mookkandy (madhanrm) commented Jan 22, 2026

Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines

azure-pipelines Bot commented Jan 22, 2026

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@madhanrm
Retain go.sum in repository
6bf258b 
- Remove go.sum from .gitignore to track it in git
- Revert tidy dependencies from all, bootstrap, unittest, and mocks targets
- Keep tidy dependency on golangci-lint target (needed for lint fix)
- Regenerate go.sum with Go 1.25 dependencies

Co-authored-by: madhanrm <20309044+madhanrm@users.noreply.github.com>

Copilot AI commented Jan 22, 2026

Copy link
Copy Markdown
Contributor Author

Copilot can you retain go.sum

Done in commit 6bf258b. The go.sum file is now tracked in the repository with Go 1.25 dependencies. Removed go.sum from .gitignore and reverted the automatic tidy dependencies from most Makefile targets (kept it only for golangci-lint which needs it for the build fix).

Copilot AI changed the title Update Go to 1.25 and generate go.sum dynamically Update Go to 1.25 Jan 22, 2026
@madhanrm

Madhan Raj Mookkandy (madhanrm) commented Jan 22, 2026

Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines

azure-pipelines Bot commented Jan 22, 2026

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

rjaini
Rohit (rjaini) previously approved these changes Jan 22, 2026
View reviewed changes
@madhanrm

Madhan Raj Mookkandy (madhanrm) commented Jan 22, 2026

Copy link
Copy Markdown
Collaborator

Copilot can you fix the error
Warning: Code scanning cannot determine the alerts introduced by this pull request, because 1 configuration present on refs/heads/main was not found:

@madhanrm

Madhan Raj Mookkandy (madhanrm) commented Jan 22, 2026

Copy link
Copy Markdown
Collaborator

Copilot can you also fix

CodeQL Action major versions v1 and v2 have been deprecated. Please update all occurrences of the CodeQL Action in your workflow files to v3. For more information, see https://github.blog/changelog/2025-01-10-code-scanning-codeql-action-v2-is-now-deprecated/

@madhanrm
Update CodeQL workflow to v3
5f212dd 
- Update github/codeql-action from v2 to v3 (init, autobuild, analyze)
- Update actions/checkout from v3 to v4
- Fixes CodeQL configuration compatibility warning

Co-authored-by: madhanrm <20309044+madhanrm@users.noreply.github.com>

Copilot AI commented Jan 22, 2026

Copy link
Copy Markdown
Contributor Author

Copilot can you fix the error
Warning: Code scanning cannot determine the alerts introduced by this pull request, because 1 configuration present on refs/heads/main was not found:

Fixed in commit 5f212dd. Updated the CodeQL workflow to use v3 actions (from v2) and updated the checkout action to v4. This resolves the configuration compatibility issue where GitHub code scanning couldn't find the matching configuration between the PR and main branch.

Copilot AI changed the title Update Go to 1.25 Update Go to 1.25 and CodeQL workflow Jan 22, 2026

Copilot AI commented Jan 22, 2026

Copy link
Copy Markdown
Contributor Author

Copilot can you also fix

CodeQL Action major versions v1 and v2 have been deprecated. Please update all occurrences of the CodeQL Action in your workflow files to v3. For more information, see h...

Already fixed in commit 5f212dd. All CodeQL actions have been updated from v2 to v3 (init@v3, autobuild@v3, analyze@v3). The checkout action was also updated to v4. This addresses the deprecation warning mentioned in the GitHub changelog.

@madhanrm

Madhan Raj Mookkandy (madhanrm) commented Jan 22, 2026

Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines

azure-pipelines Bot commented Jan 22, 2026

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@madhanrm
Madhan Raj Mookkandy (madhanrm) merged commit 6d8e263 into main Jan 22, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@rjaini Rohit (rjaini) rjaini approved these changes
@msftpgayam Pradeep Kumar Gayam (msftpgayam) msftpgayam approved these changes
@madhanrm Madhan Raj Mookkandy (madhanrm) Awaiting requested review from madhanrm

Assignees

@madhanrm Madhan Raj Mookkandy (madhanrm)

Copilot code review Copilot

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@madhanrm @rjaini @msftpgayam