search

fmjstudios / helm

🪖 A collection of MIT-licensed Helm Charts
MIT License
13 stars 13 forks source link

vaultwarden: fix various minor errors #6

Closed tctlrd closed 11 months ago

tctlrd commented 11 months ago

@FMJdev

What this PR does / why we need it

this fixes a few minor errors in the chart that were causing it to fail.

Checklist

mvprowess commented 11 months ago

Hey @tctlrd,

thank you for your contributions. Looks like I missed some stuff there 😅 These look fine so I'll merge them regardless of CI for now, they the testing workflow has a bug in it anyways.

I was gonna question about the name you put into the AUTHORS since it's not your username but I just saw it's a valid GitHub account.

Just know I plan to use the file to display contributors in the README sometime later when I get the chance. Feel free to open a separate PR to change the name if you want.

tctlrd commented 11 months ago

@FMJdev ,

Thank you for maintaining this chart and merging the fix! I used my org name since it is where the PR came from.