Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Update configuration.rst to avoid misunderstanding#16394

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
wouterj merged 1 commit intosymfony:4.4frommertingen:patch-1
Jan 19, 2022
Merged

Update configuration.rst to avoid misunderstanding#16394

wouterj merged 1 commit intosymfony:4.4frommertingen:patch-1
Jan 19, 2022

Conversation

@mertingen
Copy link
Contributor

We shouldn't deploy the ".env.local" file for the production environment variables. We just need to add the ".env" file into Git and manually create a new ".env.local" file on the production machine. In the old expression says, "deploying" and it might be a misunderstanding in this way. I changed it to "creating".

Old expression:
In production, the.env files are also parsed and loaded on each request. So
the easiest way to define env vars is by deploying a.env.local file to your
production server(s) with your production values.

Copy link
Contributor

@94noni94noni left a comment

Choose a reason for hiding this comment

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

perhaps target v5.4?
Even more, you can also reword the next paragraph and instead of usingcomposer dump-env, use the new same symfony featuresymfony/symfony#42610 :)

@mertingenmertingen changed the titleUpdate configuration.rst to aviod misunderstandingUpdate configuration.rst to avoid misunderstandingJan 15, 2022
@mertingen
Copy link
ContributorAuthor

@94noni Thanks for referring to it. Should I change the base branch from 6.0 to 5.4? In addition to that, how does that command work? Should I change the following?

From
composer dump-env prod

To
symfony console dotenv:dump prod

Or
bin/console dotenv:dump prod

We shouldn't deploy the ".env.local" file for the production environment variables. We just need to add the ".env" file into Git and manually create a new ".env.local" file on the production machine. In the old expression says, "deploying" and it might be a misunderstanding in this way. I changed it to "creating".Old expression:In production, the ``.env`` files are also parsed and loaded on each request. Sothe easiest way to define env vars is by deploying a ``.env.local`` file to yourproduction server(s) with your production values.
@wouterjwouterj changed the base branch from6.0 to4.4January 19, 2022 14:51
@wouterjwouterj merged commit885d082 intosymfony:4.4Jan 19, 2022
@wouterj
Copy link
Member

Thanks@mertingen! I've merged this in 4.4, the oldest branch that we still maintain (and this change applies to).

We're not sure how to document the newdump-env command:#15878

And to answer your question:symfony console andbin/console are effectively the same command (except thatsymfony console runs the command within the Symfony CLI env, e.g. you have access to all automatically generated env vars).

mertingen reacted with thumbs up emoji

@mertingenmertingen deleted the patch-1 branchJanuary 19, 2022 15:34
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@OskarStarkOskarStarkAwaiting requested review from OskarStark

@xabbuhxabbuhAwaiting requested review from xabbuh

1 more reviewer

@94noni94noni94noni approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@mertingen@wouterj@94noni@carsonbot

[8]ページ先頭

©2009-2025 Movatter.jp