Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Provide helmwave support #486

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

zhilyaev
Copy link

@zhilyaev zhilyaev commented May 15, 2024

Hello folks 馃憢
We have an awesome project for composing helm charts. And we would like to be part of argocd-lovely-plugin family.

@tico24
Copy link
Collaborator

tico24 commented May 15, 2024

Can you please raise an issue to help understand the use case for this addition?

Assuming we all agree it should go in like this then docs will need updating too.

I appreciate it's your tool but helmwave looks quite young and we haven't heard anyone requesting it so I'm keen to learn more.

@Joibel
Copy link
Contributor

Joibel commented May 15, 2024

I would want to have it as a variation.

The main Dockerfile is for tools which are required to make lovely work and have builtin support in the code. We want to keep this slim for security.

@zhilyaev
Copy link
Author

We can continue discussion in #489 issue if you prefer that way

@Joibel
Copy link
Contributor

Joibel commented May 15, 2024

If you can explain how it would be useable in #489, we'd consider adding it as a variation.

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

GoGitOps Review

Grade: A+ (100.0%)

Files: 21

Issues: 0

gofmt: 100%

go_vet: 100%

gocyclo: 100%

golint: 100%

ineffassign: 100%

license: 100%

misspell: 100%

This report was generated using GoGitOps.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants