Skip to content

Updated the installation instructions for Symfony 4 #8135

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

Closed
wants to merge 2 commits into from

Conversation

javiereguiluz
Copy link
Member

It's great to see how simple things are now (for example, there's nothing to say or explain about permissions because you can't have permissions problems when installing Symfony, because of the use of umask() by default).

update or replace compromised dependencies as soon as possible.
It's recommended to not submit some files (``.env``) and directories (``vendor/``,
cache, logs) to the repository, so you'll have to do the following when
installing an existing Symfony application:
Copy link
Member

Choose a reason for hiding this comment

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

this is now missing an explanation about what to do with the .env file

Copy link
Member Author

Choose a reason for hiding this comment

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

Yes ... we'll do that in the upcoming reference article about configuring Symfony apps: #7386

setup.rst Outdated

Fixing Permissions Problems
---------------------------
Once you fix all the reported issues, uninstall the requirements checker:
Copy link
Contributor

Choose a reason for hiding this comment

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

Once you've fixed

@xabbuh xabbuh added this to the 4.0 milestone Jul 11, 2017
@xabbuh
Copy link
Member

xabbuh commented Jul 11, 2017

Thank you Javier.

@xabbuh xabbuh closed this in d7dd8dd Jul 11, 2017
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.

4 participants