Skip to content

[Config] Simplify configuration env var example #17871

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

Merged
merged 1 commit into from
Feb 8, 2023

Conversation

94noni
Copy link
Contributor

@94noni 94noni commented Feb 6, 2023

Hi,

This PR supersede this one

Simplify here as the `resolve` is not needed, as reference it is documented in the dedicated doc page here
https://symfony.com/doc/current/configuration/env_var_processors.html

Targeting v5.4 instead of v4.4

friendly ping @GromNaN @javiereguiluz @HeahDude @nicolas-grekas as original reviewers

@carsonbot carsonbot added this to the 5.4 milestone Feb 6, 2023
@94noni 94noni force-pushed the env-var-config-simplify branch from 416f886 to b15ca74 Compare February 7, 2023 07:10
@carsonbot carsonbot changed the title Simplify configuration env var example [Config] Simplify configuration env var example Feb 8, 2023
@javiereguiluz javiereguiluz merged commit 4bd105a into symfony:5.4 Feb 8, 2023
@javiereguiluz
Copy link
Member

Thank you Antoine!

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

Successfully merging this pull request may close these issues.

3 participants