Skip to content

Inconsistent consolidation pattern between ci.yaml and helm-publish.yaml #19

Description

@kphunter

.github/workflows/helm-publish.yaml:R97

In ci.yaml, the push and sign operations were consolidated into a single step ("Package, push & sign chart"), removing the step id: push and the GITHUB_OUTPUT digest export. In helm-publish.yaml, the push and sign remain as separate steps, with CHART_NAME extracted independently in each (helm-publish.yaml:73 and helm-publish.yaml:97). This means helm-publish.yaml parses Chart.yaml twice. While not a bug, this is an inconsistency in approach — consider consolidating helm-publish.yaml the same way, or passing CHART_NAME via GITHUB_OUTPUT from the push step to avoid the redundant parse.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions