Skip to content

[tempo-distributed] Remove helm-docs, standardize README#120

Merged
QuentinBisson merged 3 commits intografana-community:mainfrom
TheRealNoob:remove-helm-docs/tempo-distributed
Feb 22, 2026
Merged

[tempo-distributed] Remove helm-docs, standardize README#120
QuentinBisson merged 3 commits intografana-community:mainfrom
TheRealNoob:remove-helm-docs/tempo-distributed

Conversation

@TheRealNoob
Copy link
Copy Markdown
Member

What this PR does / why we need it

Part of #107

  • Delete README.md.gotmpl
  • Rewrite README.md to standardized format with OCI + HTTP install sections, changelog link, and no auto-generated badges/Values table or version footer
  • Preserve Upgrading, Components, and Configuration sections
  • Update dependency table versions to match Chart.yaml
  • Bump version 2.3.1 → 2.3.2

Which issue this PR fixes

Special notes for your reviewer

Checklist

  • DCO signed
  • Chart Version bumped
  • Title of the PR starts with chart name (e.g. [grafana])

- Delete README.md.gotmpl
- Rewrite README.md to standardized format with OCI + HTTP install
  sections, changelog link, and no auto-generated badges/Values table
  or version footer
- Preserve Upgrading, Components, and Configuration sections
- Update dependency table versions to match Chart.yaml
- Bump version 2.3.1 → 2.3.2

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Signed-off-by: TheRealNoob <mike1118@live.com>
changes were made in grafana-community#81

Signed-off-by: TheRealNoob <mike1118@live.com>
@TheRealNoob
Copy link
Copy Markdown
Member Author

I also reverted some changes that were implemented in #81. textlint is configured to ignore comments, but helm-docs turned them into markdown tables so they had to be changed. I've only reverted the changes that specifically mentioned valid input options, as i'm not sure whether they're case-sensitive.

Copy link
Copy Markdown

@QuentinBisson QuentinBisson left a comment

Choose a reason for hiding this comment

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

LGTM

@QuentinBisson QuentinBisson merged commit 7806463 into grafana-community:main Feb 22, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants