convert to string + upgrade to 0.67.0 #10
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Motivation
The primary motivation for this pull request is to ensure that all values in the Helm chart are strings where required. This can help prevent potential issues with type mismatches in the configuration. Additionally, upgrading the chart version to 0.67.0 ensures that users are aware of this change and can benefit from any other improvements or fixes included in this new version.
Changes
Version Upgrade:
charts/hub/Chart.yaml
from0.66.0
to0.67.0
.Configuration Update:
nginx.ingress.kubernetes.io/enable-cors
incharts/hub/templates/kerberos-hub/hub-api.yaml
from a boolean to a string (true
to"true"
).Benefits
This pull request improves the overall stability and maintainability of the project by enforcing consistent data types and keeping the versioning clear and up-to-date.