Skip to content

Conversation

allanger
Copy link
Contributor

This is a PR that is supposed to cover both

I haven't added new tests, I think it might make sense, but I would first wait for a @burningalchemist opinion in the approach

@allanger allanger marked this pull request as ready for review February 20, 2024 10:14
@burningalchemist
Copy link
Owner

@allanger Thanks, I'll have a look today. 👍

@burningalchemist burningalchemist self-requested a review February 21, 2024 17:42
Copy link
Owner

@burningalchemist burningalchemist left a comment

Choose a reason for hiding this comment

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

@allanger Looks good to me! Let's give it a shot. 🚀

@burningalchemist burningalchemist self-requested a review February 21, 2024 17:59
Copy link
Owner

@burningalchemist burningalchemist left a comment

Choose a reason for hiding this comment

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

Do you mind also updating the readme for createConfig?

@allanger
Copy link
Contributor Author

Yes, sure, will try to do it within next couple of days

@burningalchemist burningalchemist self-requested a review February 24, 2024 19:35
@burningalchemist burningalchemist changed the title Add an ability to mount extra volumes feat(helm): add an ability to mount extra volumes Feb 24, 2024
@burningalchemist burningalchemist merged commit b62d4d1 into burningalchemist:master Feb 24, 2024
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.

2 participants