Skip to content

fix: clone CLI chart to /tmp to avoid submodule and copy helm docs correctly#62

Merged
dangrondahl merged 1 commit intomainfrom
fix_cli_pr_creation
Mar 10, 2026
Merged

fix: clone CLI chart to /tmp to avoid submodule and copy helm docs correctly#62
dangrondahl merged 1 commit intomainfrom
fix_cli_pr_creation

Conversation

@dangrondahl
Copy link
Collaborator

Fixes two issues with the helm docs generation step:

  • Using actions/checkout with path: cli-repo placed a nested git repo inside the docs workspace, causing create-pull-request to stage it as a submodule. Replaced with a plain git clone to /tmp/cli-repo which is outside the tracked workspace.
  • helm-docs --output-file is relative to the chart directory and cannot write outside it. Instead, generate to the default README.md in the chart dir and copy it to helm/k8s_reporter.md in the docs repo.

@dangrondahl dangrondahl merged commit 98e3c07 into main Mar 10, 2026
4 checks passed
@dangrondahl dangrondahl deleted the fix_cli_pr_creation branch March 10, 2026 23:52
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.

1 participant