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

disable additionalProperties at top level of aggregated chart #131

Closed
fstaudt opened this issue Oct 13, 2023 · 0 comments
Closed

disable additionalProperties at top level of aggregated chart #131

fstaudt opened this issue Oct 13, 2023 · 0 comments
Labels
bug Something isn't working
Milestone

Comments

@fstaudt
Copy link
Owner

fstaudt commented Oct 13, 2023

invalid keys at top-level of values.yaml is only identified.

Invalid keys at first level of values.yaml are not identified as validation failures by IntelliJ IDEA.
They are only identified as validation failures by Gradle task validateHelmValues (thanks to flag unevaluatedProperties disabled at top level).

Flag additionalProperties should be disabled at top level of aggregated chart to ensure that those invalid keys are also identified by IntelliJ IDEA.
This flag should only be removed when unevaluatedProperties is supported by IntelliJ IDEA (part of JSON schema specification 2019-09).

@fstaudt fstaudt added the bug Something isn't working label Oct 13, 2023
@fstaudt fstaudt added this to the 0.8.0 milestone Oct 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant