Skip to content
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

Bump golangci-lint to 1.50.1 #993

Merged
merged 3 commits into from
Nov 21, 2022

Conversation

@submariner-bot
Copy link

🤖 Created branch: z_pr993/skitt/golangci-lint-1.50.0

@skitt skitt added maintenance Small tasks related to ongoing maintenance lint-projects Lint consuming projects labels Oct 19, 2022
@skitt skitt force-pushed the golangci-lint-1.50.0 branch from 7acdbb8 to 82167f4 Compare October 19, 2022 10:09
@dfarrell07
Copy link
Member

Testing enabling the new linters from these versions here: dfarrell07#48

@skitt skitt force-pushed the golangci-lint-1.50.0 branch from 82167f4 to 63abdf9 Compare October 19, 2022 10:24
@skitt
Copy link
Member Author

skitt commented Oct 19, 2022

We should postpone this until after rc0, there’s quite a lot of work involved.

@dfarrell07
Copy link
Member

Maybe I can send some of the simpler fixes before the RC so we don't have to backport them? A lot of the issues seem related to spacing of nolint directives.

@skitt
Copy link
Member Author

skitt commented Oct 19, 2022

Maybe I can send some of the simpler fixes before the RC so we don't have to backport them? A lot of the issues seem related to spacing of nolint directives.

I’ve scripted that one ;-).

@skitt skitt force-pushed the golangci-lint-1.50.0 branch from 63abdf9 to 6f152ef Compare October 19, 2022 11:44
@stale
Copy link

stale bot commented Nov 12, 2022

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the wontfix This will not be worked on label Nov 12, 2022
@dfarrell07 dfarrell07 removed the wontfix This will not be worked on label Nov 14, 2022
@skitt skitt force-pushed the golangci-lint-1.50.0 branch from 6f152ef to ea3811c Compare November 14, 2022 12:56
@skitt skitt changed the title Bump golangci-lint to 1.50.0 Bump golangci-lint to 1.50.1 Nov 14, 2022
@skitt skitt force-pushed the golangci-lint-1.50.0 branch 2 times, most recently from ceb31f6 to fa91414 Compare November 15, 2022 18:16
@skitt
Copy link
Member Author

skitt commented Nov 15, 2022

@dfarrell07, @tpantelis, this is ready to go.

@tpantelis tpantelis enabled auto-merge (rebase) November 15, 2022 18:45
@skitt skitt force-pushed the golangci-lint-1.50.0 branch from fa91414 to 48473ce Compare November 16, 2022 15:07
@skitt skitt requested a review from dfarrell07 November 17, 2022 15:43
@tpantelis tpantelis merged commit ac30b24 into submariner-io:devel Nov 21, 2022
@submariner-bot
Copy link

🤖 Closed branches: [z_pr993/skitt/golangci-lint-1.50.0]

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-handled lint-projects Lint consuming projects maintenance Small tasks related to ongoing maintenance
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants