chart-releaser-action icon indicating copy to clipboard operation
chart-releaser-action copied to clipboard

Is there any way to customize pages-index-path and index-path?

Open powerfooI opened this issue 2 years ago • 2 comments

Version: [email protected]

Chart releaser has two arguments pages-index-path and index-path while it seems that they are not supported by chart-releaser-action. Is there any way to read and update index information from other path instead of default ./index.yaml?

My situation is that I use a document framework to generate static site to gh-pages, which would overwrite the whole branch including the index.yaml. So I need to update and put the index.yaml at the site's static/ folder on other branch.

$ cr index --help
Update a Helm chart repository index.yaml file based on a the
given GitHub repository's releases.

Usage:
  cr index [flags]

Flags:
   ...
  -i, --index-path string              Path to index file (default ".cr-index/index.yaml")
  -o, --owner string                   GitHub username or organization
  -p, --package-path string            Path to directory with chart packages (default ".cr-release-packages")
      --pages-branch string            The GitHub pages branch (default "gh-pages")
      --pages-index-path string        The GitHub pages index path (default "index.yaml")
   ...
Global Flags:
      --config string   Config file (default is $HOME/.cr.yaml)
$ cr version

GitVersion:  v1.6.1
GitCommit:   0102fa30dd78df0d0f8093c4607e6080972ff82d
BuildDate:   2023-10-31T07:28:18Z
GoVersion:   go1.20.10
Compiler:    gc
Platform:    darwin/arm64
License:     Apache 2.0

powerfooI avatar Mar 29 '24 07:03 powerfooI

Another motivation for this is when relying on the GitHub pages feature, there is an option to select /root or /docs as a source for the GitHub pages. While things work smoothly for /root, creating a working repository with /docs fails.

masayag avatar Mar 31 '24 18:03 masayag

Thank you @powerfooI. I missed the reference to your PR and published the same fix. I wish to see this one getting merged.

masayag avatar Mar 31 '24 19:03 masayag