Fix rules_helm dependencies #3001
Conversation
|
Hello @abrisco, modules you maintain (rules_helm) have been updated in this PR. Please review the changes. |
|
I think this change can be closed in favor of #3016 |
|
@abrisco it would be nice if there are not random broken versions on BCR. Yanking versions is also bad as this also breaks builds which might only use other functionality which works. So I would prefer to upload this additionally (especially as it doesn't even change the behavior of the code). |
|
Sounds good to me! |
|
Should users just switch to the latest version? #3054 |
mering
force-pushed
the
rules-helm-fix-deps
branch
from
b530121
to
5acc712
Compare
bazel-io
dismissed stale reviews from abrisco and themself
Require module maintainers' approval for newly pushed changes.
As always, it is recommended to use the latest version but in case people want to use an older version, they should find a working version on BCR. |
rules_oci is loaded by a non dev_dependencies module extension and therefore musn't be a dev_dependency itself
mering
force-pushed
the
rules-helm-fix-deps
branch
from
5acc712
to
6b5d0ee
Compare
rules_oci is loaded by a non dev_dependencies module extension and therefore musn't be a dev_dependency itself.
Proposed an upstream fix for future versions in abrisco/rules_helm#107.