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.

Sign up for GitHub

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

Jump to bottom

[NO-ISSUE] fix: remove database_url check in chart #35

Merged

GuillaumeDecMeetsMore merged 2 commits into master from guillaume/fix/chart-remove-database-url-check

Aug 29, 2024

Collaborator

GuillaumeDecMeetsMore commented Aug 29, 2024 •

edited

Loading

Changed

Remove check on DATABASE_URL
- We allow all envs now ([NO-ISSUE] fix(helm): allow all "envs" values + update chart version #34)
  - TODO (later): add explaantion of expected env vars in chart's folder and in values.yml
Update chart version for triggering new deployment

GuillaumeDecMeetsMore added 2 commits

August 29, 2024 15:49


          fix: remove database_url check in chart

e0032da


          chore: update chart version

a0e30f9

GuillaumeDecMeetsMore self-assigned this

GuillaumeDecMeetsMore marked this pull request as ready for review

August 29, 2024 06:51

GuillaumeDecMeetsMore requested review from mm-zacharydavison and colachg

August 29, 2024 06:51

GuillaumeDecMeetsMore enabled auto-merge (squash)

August 29, 2024 06:52

colachg approved these changes

View reviewed changes

GuillaumeDecMeetsMore merged commit 00efa39 into master

2 checks passed

GuillaumeDecMeetsMore deleted the guillaume/fix/chart-remove-database-url-check branch

August 29, 2024 06:53

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet