Skip to content

Fix wsl_v5 linter warnings reported in golangci-lint 2.9.0#1281

Merged
tpantelis merged 1 commit intosubmariner-io:develfrom
tpantelis:adj_lint_2.9.0
Feb 15, 2026
Merged

Fix wsl_v5 linter warnings reported in golangci-lint 2.9.0#1281
tpantelis merged 1 commit intosubmariner-io:develfrom
tpantelis:adj_lint_2.9.0

Conversation

@tpantelis
Copy link
Contributor

@tpantelis tpantelis commented Feb 12, 2026

Add blank lines before return and go statements to satisfy wsl_v5 whitespace requirements.

Summary by CodeRabbit

  • Tests
    • Applied minor formatting improvements to test files for consistency.

Add blank lines before return and go statements to satisfy
wsl_v5 whitespace requirements.

Signed-off-by: Tom Pantelis <tompantelis@gmail.com>
@submariner-bot
Copy link
Contributor

🤖 Created branch: z_pr1281/tpantelis/adj_lint_2.9.0

@coderabbitai
Copy link

coderabbitai bot commented Feb 12, 2026

Walkthrough

Two test files received minor formatting adjustments: blank lines were added in test setup sections for improved readability. No functional, logic, or control flow changes were introduced.

Changes

Cohort / File(s) Summary
Test Formatting
pkg/resource/interface_test.go, pkg/workqueue/queue_test.go
Added blank lines in test setup sections for improved readability. No logic or behavior changes.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Suggested reviewers

  • skitt
  • vthapar
  • sridhargaddam
  • Oats87
🚥 Pre-merge checks | ✅ 3 | ❌ 1
❌ Failed checks (1 inconclusive)
Check name Status Explanation Resolution
Actionable Comments Resolved ❓ Inconclusive Cannot verify actionable comments without access to the actual GitHub PR comments and review feedback for PR #1281. Access the GitHub PR directly or provide the review comments to verify all actionable items have been addressed.
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main change: adding blank lines to fix wsl_v5 linter warnings from golangci-lint 2.9.0, which matches the changeset.
Merge Conflict Detection ✅ Passed ✅ No merge conflicts detected when merging into devel
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@tpantelis tpantelis enabled auto-merge (rebase) February 12, 2026 19:53
@tpantelis tpantelis disabled auto-merge February 15, 2026 14:13
@tpantelis tpantelis enabled auto-merge (rebase) February 15, 2026 14:13
@tpantelis tpantelis merged commit a3e194f into submariner-io:devel Feb 15, 2026
21 checks passed
@submariner-bot
Copy link
Contributor

🤖 Closed branches: [z_pr1281/tpantelis/adj_lint_2.9.0]

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