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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow alpha config to come from a template #152

Merged

Conversation

mkjpryor
Copy link
Contributor

@mkjpryor mkjpryor commented Jun 7, 2023

Fixes #151

@pierluigilenoci
Copy link
Contributor

Thank you so much for your PR.

To approve this PR, the Chart version bump and the modification to the README.md to explain how to use the parameter are missing.
Could you please add them?

@mkjpryor
Copy link
Contributor Author

mkjpryor commented Jun 7, 2023

@pierluigilenoci I realised I didn't add the documentation...! Sorry!

For the version bump, are you expecting a minor or patch version bump? This is obviously a backwards compatible change.

@mkjpryor
Copy link
Contributor Author

mkjpryor commented Jun 7, 2023

@pierluigilenoci

I added the docs and version bump. I assumed a minor version bump, since this is a feature addition and not just a fix.

@pierluigilenoci
Copy link
Contributor

That's fine.
Thanks again.

@pierluigilenoci pierluigilenoci merged commit 518dc3c into oauth2-proxy:main Jun 7, 2023
1 check passed
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.

Allow templating of alpha config
2 participants