Skip to content

Document how to resolve env var at compile time #21223

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
Jul 22, 2025

Conversation

nikophil
Copy link
Contributor

@nikophil nikophil commented Jul 20, 2025

see https://symfony-devs.slack.com/archives/C3EQ7S3MJ/p1752843597330219

Resolving env var at compile is a unusual case, but this could be helpful from time to time.

This piece of information is hard to find (I've found it here), and it could be interesting to have it documented, with a meaningful warning.

@nikophil nikophil force-pushed the resoving-env-var-compile-time branch from 9bd0c03 to 44d8911 Compare July 20, 2025 15:40
Comment on lines +955 to +920
Resolving Environment Variable At Compile Time
----------------------------------------------

Although environment variables are usually a runtime concept, you can also resolve them
:ref:`at compile time <resolving-env-vars-at-compile-time>`.
Copy link
Contributor Author

@nikophil nikophil Jul 20, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

every search in google about "symfony env var" lead to this page, so it seems useful to have link here to the other page

@javiereguiluz javiereguiluz changed the base branch from 7.3 to 6.4 July 22, 2025 14:42
@javiereguiluz javiereguiluz requested a review from xabbuh as a code owner July 22, 2025 14:42
@javiereguiluz javiereguiluz force-pushed the resoving-env-var-compile-time branch from 44d8911 to 4aaca90 Compare July 22, 2025 14:42
@javiereguiluz javiereguiluz merged commit 4f8ddfe into symfony:6.4 Jul 22, 2025
3 checks passed
@javiereguiluz
Copy link
Member

Merged! Thanks Nicolas.

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