Skip to content

In the helm_lib_module_image helper, we need to insert the lowerCamelCase module name - #142

Merged
ldmonster merged 2 commits into
mainfrom
lowercamelcase-helm-lib-fix
Oct 17, 2025
Merged

ldmonster merged 2 commits into
mainfrom
lowercamelcase-helm-lib-fix

Conversation

@RottenRat

@RottenRat RottenRat commented Oct 16, 2025 •

Copy link
Copy Markdown
Contributor

In the helm_lib_module_image helper, you need to insert the module name in lowerCamelCase, not kebab-case.

Sinelnikov Michail
tst
Signed-off-by: Sinelnikov Michail <mikhail.sinelnikov@flant.com>
@RottenRat RottenRat self-assigned this Oct 16, 2025
Signed-off-by: Sinelnikov Michail <mikhail.sinelnikov@flant.com>
@RottenRat
RottenRat marked this pull request as ready for review October 16, 2025 12:38
@ldmonster ldmonster added the bug Something isn't working label Oct 17, 2025
@ldmonster
ldmonster merged commit ab830b4 into main Oct 17, 2025
4 checks passed
@ldmonster
ldmonster deleted the lowercamelcase-helm-lib-fix branch October 17, 2025 00:05
ldmonster added a commit that referenced this pull request Oct 24, 2025
…werCamelCase module name (#142)"

This reverts commit ab830b4.

Signed-off-by: Pavel Okhlopkov <pavel.okhlopkov@flant.com>
ldmonster added a commit that referenced this pull request Oct 24, 2025
…werCamelCase module name (#142)" (#149)

This reverts commit ab830b4.

Signed-off-by: Pavel Okhlopkov <pavel.okhlopkov@flant.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants