Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Jan 21, 2026

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.

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 requested a review from madhanrm January 21, 2026 05:41
Copilot AI and others added 2 commits January 21, 2026 05:56
Co-authored-by: madhanrm <20309044+madhanrm@users.noreply.github.com>
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 madhanrm marked this pull request as ready for review January 21, 2026 06:03
@madhanrm
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@madhanrm
Copy link
Collaborator

@copilot can you fix the lint build error

- 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>
Copy link
Contributor Author

Copilot AI commented Jan 21, 2026

@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.

Copilot AI requested a review from madhanrm January 21, 2026 23:35
@madhanrm
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@madhanrm
Copy link
Collaborator

@copilot can you retain go.sum

@madhanrm
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

- 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>
Copy link
Contributor Author

Copilot AI commented Jan 22, 2026

@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
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

rjaini
rjaini previously approved these changes Jan 22, 2026
@madhanrm
Copy link
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
Copy link
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/

- 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>
Copy link
Contributor Author

Copilot AI commented Jan 22, 2026

@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
Copy link
Contributor Author

Copilot AI commented Jan 22, 2026

@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
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@madhanrm 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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants