Skip to content

golangci-lint v2 - #623

Merged
l0wl3vel merged 5 commits into
mainfrom
feat/golangci-lint-v2
Aug 14, 2026
Merged

golangci-lint v2#623
l0wl3vel merged 5 commits into
mainfrom
feat/golangci-lint-v2

Conversation

@l0wl3vel

@l0wl3vel l0wl3vel commented Aug 3, 2026

Copy link
Copy Markdown
Collaborator
  • Upgraded to golangci-lint v2, since go 1.26 required for postgres-operator v2 is not compatible with golangci-lint v1
  • Migrated the old linter presets
  • added more style linters (only for new commits)
  • applied autofixes, where available

blocks: #622

Signed-off-by: Benjamin Ritter <benjamin.ritter@x-cellent.com>
@l0wl3vel
l0wl3vel requested a review from TLINDEN August 3, 2026 16:09
@l0wl3vel l0wl3vel added enhancement New feature or request dependencies Pull requests that update a dependency file labels Aug 3, 2026
@l0wl3vel
l0wl3vel force-pushed the feat/golangci-lint-v2 branch from d03b1e9 to b34ae3a Compare August 3, 2026 16:21
Signed-off-by: Benjamin Ritter <benjamin.ritter@x-cellent.com>
Signed-off-by: Benjamin Ritter <benjamin.ritter@x-cellent.com>
Signed-off-by: Benjamin Ritter <benjamin.ritter@x-cellent.com>

@TLINDEN TLINDEN left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Everything else is good to go.

Comment thread controllers/postgres_controller.go Outdated
Signed-off-by: Benjamin Ritter <benjamin.ritter@x-cellent.com>
@l0wl3vel
l0wl3vel requested a review from TLINDEN August 14, 2026 09:14

@TLINDEN TLINDEN left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

ack.

@l0wl3vel
l0wl3vel merged commit 04cd457 into main Aug 14, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants