Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(argocd-apps): don't render empty app description #2911

Merged
merged 3 commits into from
Sep 10, 2024

Conversation

reegnz
Copy link
Contributor

@reegnz reegnz commented Sep 4, 2024

Rendering empty description leads to failed validation when using kubeconform.

Signed-off-by: Zoltán Reegn [email protected]

Checklist:

  • I have bumped the chart version according to versioning
  • I have updated the documentation according to documentation
  • I have updated the chart changelog with all the changes that come with this pull request according to changelog.
  • Any new values are backwards compatible and/or have sensible default.
  • I have signed off all my commits as required by DCO.
  • My build is green (troubleshooting builds).

@reegnz reegnz force-pushed the fix_rendering_empty_description branch from 7061f3f to d7faef4 Compare September 4, 2024 16:36
@github-actions github-actions bot added size/S and removed size/XS labels Sep 4, 2024
yu-croco
yu-croco previously approved these changes Sep 4, 2024
@yu-croco
Copy link
Collaborator

yu-croco commented Sep 4, 2024

Hi @reegnz , thank you for your PR.
Please fix lint ( https://github.com/argoproj/argo-helm/actions/runs/10705865587/job/29697905648?pr=2911 ), otherwise LGTM.

@yu-croco yu-croco dismissed their stale review September 5, 2024 08:40

need to pass CI

charts/argocd-apps/Chart.yaml Outdated Show resolved Hide resolved
Signed-off-by: Marco Maurer (-Kilchhofer) <[email protected]>
@yu-croco yu-croco merged commit a344d64 into argoproj:main Sep 10, 2024
6 checks passed
@reegnz reegnz deleted the fix_rendering_empty_description branch September 11, 2024 08:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants