Skip to content

bump to golangci-lint v2#55828

Merged
istio-testing merged 1 commit intoistio:masterfrom
zirain:adopt-golangci-v2
Apr 10, 2025
Merged

bump to golangci-lint v2#55828
istio-testing merged 1 commit intoistio:masterfrom
zirain:adopt-golangci-v2

Conversation

@zirain
Copy link
Copy Markdown
Member

@zirain zirain commented Apr 7, 2025

Please provide a description of this PR:

wait istio/tools#3180 and istio/common-files#1167

@zirain zirain added the do-not-merge Block automatic merging of a PR. label Apr 7, 2025
@zirain zirain requested a review from a team as a code owner April 7, 2025 04:56
@istio-policy-bot istio-policy-bot added area/test and release release-notes-none Indicates a PR that does not require release notes. labels Apr 7, 2025
@istio-testing istio-testing added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Apr 7, 2025
Copy link
Copy Markdown
Member

@howardjohn howardjohn left a comment

Choose a reason for hiding this comment

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

This is only changing the override, not the base file as well. Once the base file and binary comes in, there will be a ton of lint errors from when I tried it?

@zirain zirain force-pushed the adopt-golangci-v2 branch from a29cae8 to 2db9d3b Compare April 8, 2025 01:51
@istio-testing istio-testing added size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. and removed size/L Denotes a PR that changes 100-499 lines, ignoring generated files. labels Apr 8, 2025
@zirain
Copy link
Copy Markdown
Member Author

zirain commented Apr 8, 2025

@howardjohn we're good to go?

Copy link
Copy Markdown
Member

@howardjohn howardjohn left a comment

Choose a reason for hiding this comment

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

Nice, LGTM

@zirain
Copy link
Copy Markdown
Member Author

zirain commented Apr 8, 2025

Nice, LGTM

you're right, we can remove .golangci-format.yaml now.

@zirain zirain force-pushed the adopt-golangci-v2 branch from 4813592 to 0176292 Compare April 10, 2025 13:30
@istio-testing istio-testing added size/L Denotes a PR that changes 100-499 lines, ignoring generated files. and removed size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. labels Apr 10, 2025
@zirain zirain removed the do-not-merge Block automatic merging of a PR. label Apr 10, 2025
@zirain zirain requested a review from howardjohn April 10, 2025 13:33
@istio-testing istio-testing merged commit bf744c7 into istio:master Apr 10, 2025
29 checks passed
@zirain zirain deleted the adopt-golangci-v2 branch April 10, 2025 15:25
deveshdama pushed a commit to deveshdama/istio that referenced this pull request Apr 18, 2025
fjglira pushed a commit to fjglira/istio that referenced this pull request Sep 26, 2025
* upstream/master:
  Automator: update ztunnel@master in istio/istio@master (istio#55888)
  Automator: update proxy@master in istio/istio@master (istio#55882)
  golangci-lint v2 (istio#55828)
  Automator: update proxy@master in istio/istio@master (istio#55876)
  Automator: update common-files@master in istio/istio@master (istio#55878)
  addons: Bump addons version (istio#55874)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/test and release release-notes-none Indicates a PR that does not require release notes. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants