Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

chore(deps): Bump golangci-lint from v1.60.3 to v1.61.0 #15856

Merged
merged 2 commits into from
Sep 11, 2024

Conversation

zak-pawel
Copy link
Collaborator

Summary

Checklist

  • No AI generated code was used in this PR

@telegraf-tiger
Copy link
Contributor

telegraf-tiger bot commented Sep 9, 2024

@srebhan srebhan changed the title chore(linters): Bump golangci-lint to v1.61.0 chore(linters): Bump golangci-lint from v1.60.3 to v1.61.0 Sep 10, 2024
@srebhan srebhan changed the title chore(linters): Bump golangci-lint from v1.60.3 to v1.61.0 chore(deps): Bump golangci-lint from v1.60.3 to v1.61.0 Sep 10, 2024
@srebhan srebhan added the dependencies Pull requests that update a dependency file label Sep 10, 2024
@srebhan
Copy link
Member

srebhan commented Sep 10, 2024

@zak-pawel did you see

golangci/golangci-lint@bf4a66a gosec: disable G407 (golangci/golangci-lint#4983)

in the release notes? IIRC we enabled this linter just recently didn't we?

@zak-pawel
Copy link
Collaborator Author

@srebhan I saw that.

IIRC we enabled this linter just recently didn't we?

We haven't even started discussing whether we want to turn it on yet :)
image
#12957

@srebhan
Copy link
Member

srebhan commented Sep 10, 2024

Oh I confused it with the MD5 one, just ignore what I said. ;-)

Copy link
Member

@srebhan srebhan left a comment

Choose a reason for hiding this comment

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

Thanks @zak-pawel!

@srebhan srebhan added the ready for final review This pull request has been reviewed and/or tested by multiple users and is ready for a final review. label Sep 10, 2024
@srebhan srebhan assigned DStrand1 and unassigned srebhan Sep 10, 2024
@DStrand1 DStrand1 merged commit c6f615d into influxdata:master Sep 11, 2024
31 checks passed
@github-actions github-actions bot added this to the v1.32.1 milestone Sep 11, 2024
srebhan pushed a commit that referenced this pull request Oct 7, 2024
asaharn pushed a commit to asaharn/telegraf that referenced this pull request Oct 16, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
chore dependencies Pull requests that update a dependency file linter ready for final review This pull request has been reviewed and/or tested by multiple users and is ready for a final review.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants