Skip to content

fix: adding Helm nil guards for optional values - #5112

Merged
jschumacher-wire merged 2 commits into
developfrom
fixing_helm_nil_pointers
Mar 11, 2026
Merged

fix: adding Helm nil guards for optional values#5112
jschumacher-wire merged 2 commits into
developfrom
fixing_helm_nil_pointers

Conversation

@jschumacher-wire

Copy link
Copy Markdown
Contributor

Checklist

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

@jschumacher-wire
jschumacher-wire requested review from a team as code owners March 11, 2026 09:29
@zebot zebot added the ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist label Mar 11, 2026

@battermann battermann left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I have one comment, but I am unsure if this is relevant.

Comment thread charts/wire-server/templates/galley/secret.yaml Outdated
@jschumacher-wire
jschumacher-wire merged commit 80ea4c1 into develop Mar 11, 2026
8 of 10 checks passed
@jschumacher-wire
jschumacher-wire deleted the fixing_helm_nil_pointers branch March 11, 2026 11:03
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