Skip to content

WPB-23896: fix trivial SonarQube errors#5280

Open
blackheaven wants to merge 4 commits into
developfrom
gdifolco/WPB-23896-sonarqube-blockers-high-helm-restart
Open

WPB-23896: fix trivial SonarQube errors#5280
blackheaven wants to merge 4 commits into
developfrom
gdifolco/WPB-23896-sonarqube-blockers-high-helm-restart

Conversation

@blackheaven

Copy link
Copy Markdown
Contributor

https://wearezeta.atlassian.net/browse/WPB-23896

Checklist

  • Add a new entry in an appropriate subdirectory of changelog.d
  • Read and follow the PR guidelines

@blackheaven blackheaven requested review from a team as code owners June 19, 2026 13:52
@zebot zebot added the ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist label Jun 19, 2026

@akshaymankar akshaymankar left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

IMO the whole errors should go to stderr.

Comment thread hack/bin/generate-clients.sh Outdated
Comment thread hack/bin/integration-setup-federation.sh Outdated
Co-authored-by: Akshay Mankar <axeman@mailbox.org>
@blackheaven blackheaven requested a review from akshaymankar June 22, 2026 07:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants