8000
Skip to content

feat(repo): add --no-headers option to 'helm repo list'#31448

Merged
mattfarina merged 2 commits intohelm:mainfrom
laervn:feat/add-no-headers-option
Dec 17, 2025
Merged

feat(repo): add --no-headers option to 'helm repo list'#31448
mattfarina merged 2 commits intohelm:mainfrom
laervn:feat/add-no-headers-option

Conversation

@laervn
Copy link
Copy Markdown
Contributor
@laervn laervn commented Nov 1, 2025 •
edited
Loading

This adds a --no-headers flag to the 'helm repo list' command, allowing users to suppress table headers in the output. Useful for scripting and automation.

What this PR does / why we need it:

  • Closes: Add --no-headers flag to helm repo list #31449"
  • Introduces a boolean flag --no-headers to helm repo list.
  • Default behavior remains unchanged (headers are shown).
  • Makes output easier to parse in scripts or automation.
  • Includes unit test to verify no-header output.

Special notes for your reviewer:

If applicable:

  • this PR contains user facing changes (the docs needed label should be applied if so)
  • this PR contains unit tests
  • this PR has been tested for backwards compatibility

@laervn
feat(repo): add --no-headers option to 'helm repo list'
6ef79bb 
This adds a --no-headers flag to the 'helm repo list' command,
allowing users to suppress table headers in the output.
Useful for scripting and automation.

Signed-off-by: Paul Van Laer <paul.van.laer1@gmail.com>
@pull-request-size pull-request-size bot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Nov 1, 2025
@laervn laervn mentioned this pull request Nov 1, 2025
Closed
@benoittgt
Copy link
Copy Markdown
Contributor
benoittgt commented Nov 1, 2025

I think it is a good idea. LGTM

@TerryHowe
Copy link
Copy Markdown
Contributor
TerryHowe commented Nov 4, 2025

Could you add a "Closes: #31449" to your description?

TerryHowe
TerryHowe approved these changes Nov 4, 2025
View reviewed changes
Copy link
Copy Markdown
Contributor
@TerryHowe TerryHowe 8000 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

@scottrigby scottrigby added this to the 4.1.0 milestone Nov 7, 2025
@TerryHowe TerryHowe added the Has One Approval This PR has one approval. It still needs a second approval to be merged. label Dec 13, 2025
mattfarina
mattfarina approved these changes Dec 17, 2025
View reviewed changes
Copy link
Copy Markdown
Collaborator
@mattfarina mattfarina left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@mattfarina mattfarina merged commit 2108bc5 into helm:main Dec 17, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mattfarina mattfarina mattfarina approved these changes

@TerryHowe TerryHowe TerryHowe approved these changes

@banjoh banjoh banjoh approved these changes

Assignees

No one assigned

Labels

feature Has One Approval This PR has one approval. It still needs a second approval to be merged. size/S Denotes a PR that changes 10-29 lines, ignoring generated files.

Projects

None yet

Milestone

4.1.0

Development

Successfully merging this pull request may close these issues.

Add --no-headers flag to helm repo list

6 participants

@laervn @benoittgt @TerryHowe @mattfarina @banjoh @scottrigby
0