Support sub chart dir

[tuannvm]

Why?

• This change in PR will bring sub-directory support for charts_dir. Let's go through this example:

.
├── ops
│   ├── helm
│   │   ├── .helmignore
│   │   ├── chart1
│   │   ├── chart2
│   │   ├── chart3
│   │   ├── chart4
│   │   └── chart5

So, the charts_dir would be ops/helm

• Before:

cut -d '/' -f "1,2" <<< "ops/helm/chart1" ## -----> ops/helm --> no chart found

• After:

cut -d '/' -f "1-" <<< "ops/helm/chart1" ## -----> ops/helm/chart1 --> chart found

Signed-off-by: Tommy Nguyen tuannvm@hotmail.com

[unguiculus]

Thanks for the PR. I think, this needs some more thought. The problem is that this will also detect local chart dependencies which not desireable. I'm not really sure if we should support your use case. Have you tried changing the working directory? Maybe that would work in your case.

[gabriellemadden]

Have you tried changing the working directory? Maybe that would work in your case.

Changing working directory does not fix the issue, as the output of git diff (called here: https://github.com/helm/chart-releaser-action/blob/master/cr.sh#L200) still returns a path from the root of the git repo no matter which subdirectory you're in. This PR handles nested charts_dir nicely: #47 - are there any plans to merge this?

[unguiculus]

Closing this in favor of #47.