ct lint: add --ignore-ci-changes flag to skip version bump#816
Open
ct lint: add --ignore-ci-changes flag to skip version bump#816
--ignore-ci-changes flag to skip version bump#816Conversation
When only files in a chart's 'ci' directory are modified, the version increment check can be skipped during linting. This is useful because ci/ contains test values files that don't warrant a chart version bump. The flag is available for 'ct lint' and 'ct lint-and-install'. Charts with CI-only changes are still processed by 'ct install'. Signed-off-by: Alexander Bayandin <alexander@bayandin.dev>
Signed-off-by: Alexander Bayandin <alexander@bayandin.dev>
bayandin
force-pushed
the
ignore-ci-changes
branch
from
6c34098 to
5151f9b
Compare
bayandin
changed the title
--ignore-ci-changes flag to skip version bump
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
What this PR does / why we need it:
Adds
--ignore-ci-changesflag toct lintandct lint-and-install. When enabled, charts where only files in theci/directory changed skip the version increment check — all other lint steps (schema validation, YAML linting,helm lint) still run normally. Charts with CI-only changes are also still fully processed byct installso new test values files get exercised.The
ci/directory contains*-values.yamlfiles used for CI testing. These files are not included in the packaged chart (helm package), so changes to them are no-op from the chart's perspective and there should be a way to skip the version bump requirement.Which issue this PR fixes: N/A
Special notes for your reviewer:
Developed with claude-4.6-opus (Anthropic) assistance via Claude Code CLI.