Skip to content

[maintenance] bump golangci-lint to latest#1802

Merged
ytsarev merged 1 commit intomasterfrom
bump-golangci-lint
Jan 20, 2025
Merged

[maintenance] bump golangci-lint to latest#1802
ytsarev merged 1 commit intomasterfrom
bump-golangci-lint

Conversation

@kuritka
Copy link
Collaborator

@kuritka kuritka commented Jan 15, 2025

bumping golangci-lint to latest

log.Info().
Str("HeartbeatTXTName", heartbeatTXTName).
Msg("Creating split brain TXT record")
//nolint:gosec
Copy link
Member

Choose a reason for hiding this comment

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

@kuritka, can we document the skipping of this linter? Without additional comment, it looks slightly insecure :D

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Hi @ytsarev , sure, amended

Signed-off-by: Michal Kuritka <kuritka@gmail.com>
@kuritka kuritka force-pushed the bump-golangci-lint branch from 14fa3a0 to f265ad6 Compare January 20, 2025 09:11
@kuritka kuritka requested a review from ytsarev January 20, 2025 09:22
@ytsarev ytsarev merged commit d7e1527 into master Jan 20, 2025
22 checks passed
@ytsarev ytsarev deleted the bump-golangci-lint branch January 20, 2025 10:43
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.

2 participants