-
Notifications
You must be signed in to change notification settings - Fork 242
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
chore(deps): update dependencies in release/v1.4 #3493
base: release/v1.4
Are you sure you want to change the base?
Conversation
There was a problem hiding this 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 updates the project’s dependency toolchain to Go 1.24 while refreshing dependency versions managed by Dependabot and updating linting configurations.
- Update Go versions in GitHub workflow configs
- Update golangci-lint configuration with modified linter entries
Reviewed Changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.
File | Description |
---|---|
.github/workflows/crdgen.yaml | Update Go versions used for generating CRDs |
.github/workflows/golangci.yaml | Update Go versions used for linting in workflows |
.golangci.yml | Add new linters and replace an existing linter entry |
/azp run Azure Container Networking PR |
Azure Pipelines successfully started running 1 pipeline(s). |
I'm not sure what the effect of the go.mod saying 1.23 (toolchain 1.24) and the builder images using MSFT Go 1.23 will be...I think it doesn't actually upgrade us to 1.24 |
hmm maybe you are right, ill just bump everything to 1.24 then |
linter is not recognizing NPM embedded structs coming from a different module 😕 |
/azp run Azure Container Networking PR |
Azure Pipelines successfully started running 1 pipeline(s). |
/azp run Azure Container Networking PR |
Azure Pipelines successfully started running 1 pipeline(s). |
/azp run Azure Container Networking PR |
Azure Pipelines successfully started running 1 pipeline(s). |
/azp run Azure Container Networking PR |
Azure Pipelines successfully started running 1 pipeline(s). |
/azp run Azure Container Networking PR |
Azure Pipelines successfully started running 1 pipeline(s). |
Reason for Change:
Issue Fixed:
Requirements:
Notes: