-
Notifications
You must be signed in to change notification settings - Fork 48
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’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix/bash env test #1
base: master
Are you sure you want to change the base?
Fix/bash env test #1
Conversation
Thanks for the contribution! Before we can merge this, we need @msutter to sign the Sonatype Contributor License Agreement. |
@dtshepherd wdyt about this one? |
Sorry @DarthHater, I've just been absolutely slammed at work lately. This PR looks good and I see an overlapping PR #2. I like this one more because it keeps the |
@msutter Did you want to make the change I suggested above so we can get this merged? (sorry for being slow to respond, slammed in my day job...) |
@dtshepherd @DarthHater any updates on this? After the changes suggested in #1 (comment) it should be straightforward to merge, right? |
Yeah, should be. If I find some spare cycles this week, I can fork this change and apply my suggested fix. |
Any news here? I just upgraded my Nexus to get native Helm chart support, but I got stuck here. :) |
Using the plugin in some docker images produce an 'unbound variable' error by pushing charts to the nexus repository.
You can reproduce this issue by using the 'devth/helm' docker image.
This pull request makes the following changes:
(http://wiki.bash-hackers.org/syntax/pe#use_a_default_value)