Skip to content

fix: rewrite all app-of-apps to use valuesObject instead of values#1376

Draft
hairmare wants to merge 1 commit intomainfrom
fix/app-of-apps/use-valuesObject
Draft

fix: rewrite all app-of-apps to use valuesObject instead of values#1376
hairmare wants to merge 1 commit intomainfrom
fix/app-of-apps/use-valuesObject

Conversation

@hairmare
Copy link
Copy Markdown
Contributor

@hairmare hairmare commented Mar 7, 2025

Description

This makes it easier to use tools that interact with YAML for the *-apps charts. We've adopted the valuesObject change in our practice for a while, this applies that change under the hood as well.

Issues

Checklist

  • This PR contains a description of the changes I'm making
  • I updated the version in Chart.yaml
  • I updated the changelog with an artifacthub.io/changes annotation in Chart.yaml, check the example in the documentation.
  • I updated applicable README.md files using pre-commit run
  • I documented any high-level concepts I'm introducing in docs/
  • CI is currently green and this is ready for review
  • I am ready to test changes after they are applied and released

@github-actions github-actions Bot added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label Mar 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/M Denotes a PR that changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant