More than 1 subchart (deps) are ignored

[BigForNothing]

I tried to get a helm chart to build with subcharts, but there doesn't seem to be a way to get that to work. I tried to use the deps option of helm_package, and that works but only for one dep. If I specify two or more, only the most recent chart is included. The one chart that is included, everything is proper.

• I'm not sure if the chart.metadata.json is supposed to list anything about the subcharts, but it does not.
• the deps_manifest.json does list all the charts

I think this is a bug in the deps handling

[abrisco]

That would definitely be a bug! I'd be happy to review a PR to fix it 😄

[abrisco]

I added some tests in #97 which shows dependencies are working correctly, one thing that could be improved is that adding something to the deps attribute still requires you to add the dependency to your Chart.yaml. The rules should probably be updated to error if these two lists are not in sync. Can you double check your Chart.yaml is up to date?

[purkhusid]

This happens when you have dependencies that have the same target name. E.g.

helm_package(
    deps = ["//a/b/c:yo", "//x/y/z:yo"]
)

This is fixed in #123