Skip to content

EPMDEDP-17244: fix: align scaffolded helm chart README with helm-docs - #302

Merged
SergK merged 1 commit into
masterfrom
fix-helm-docs
Jul 30, 2026
Merged

EPMDEDP-17244: fix: align scaffolded helm chart README with helm-docs#302
SergK merged 1 commit into
masterfrom
fix-helm-docs

Conversation

@SergK

@SergK SergK commented Jul 30, 2026

Copy link
Copy Markdown
Member

Charts scaffolded with the Envoy Gateway (httproute) exposure failed the helm-docs check on the first build pipeline: the README template listed httproute keys out of alphabetical order and documented gateway.name/namespace as separate rows instead of the single object row helm-docs generates from the values doc comment. Reorder the httproute block before image keys, collapse the gateway rows into one object entry, and keep ingress rows only for non-envoy scaffolds.

Charts scaffolded with the Envoy Gateway (httproute) exposure failed the helm-docs check on the
first build pipeline: the README template listed httproute keys out of alphabetical order and
documented gateway.name/namespace as separate rows instead of the single object row helm-docs
generates from the values doc comment. Reorder the httproute block before image keys, collapse
the gateway rows into one object entry, and keep ingress rows only for non-envoy scaffolds.

Signed-off-by: Sergiy Kulanov <sergiy_kulanov@epam.com>
@SergK
SergK requested a review from a team as a code owner July 30, 2026 08:35
@epmd-edp

Copy link
Copy Markdown
Collaborator

Pipeline review-codebase-operator-master-t95ch ✅ Passed

Status Task Duration
github-set-pending-status 6s
fetch-repository 9s
init-values 7s
get-cache 2m8s
commit-validate 7s
dockerfile-lint 7s
helm-lint 7s
helm-docs 1m10s
build 45s
sonar 18s
save-cache 8s
buildkit-build 18s
github-report-pipeline-status 6s

@SergK
SergK merged commit 89a95de into master Jul 30, 2026
6 checks passed
@SergK
SergK deleted the fix-helm-docs branch July 30, 2026 08:51
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