Skip to content
This repository was archived by the owner on Nov 14, 2019. It is now read-only.

Update README.md #122

Merged
merged 1 commit into from
Mar 26, 2015
Merged

Update README.md #122

merged 1 commit into from
Mar 26, 2015

Conversation

JulienItard
Copy link
Contributor

Update Linux installation step

Update Linux installation step
@stof
Copy link
Member

stof commented Mar 26, 2015

the advantage of the previous instruction is that it does not run curl as root

@JulienItard
Copy link
Contributor Author

Yes i know but i think that we should have the same installation step here and on the official website.

@javiereguiluz
Copy link
Member

@JulienItard thanks for this pull request. Indeed it's a good idea to have the same installation instructions everywhere. Thanks!

@javiereguiluz javiereguiluz merged commit f32349c into symfony:master Mar 26, 2015
javiereguiluz added a commit that referenced this pull request Mar 26, 2015
This PR was merged into the 1.0-dev branch.

Discussion
----------

Update README.md

Update Linux installation step

Commits
-------

f32349c Update README.md
@JulienItard JulienItard deleted the patch-1 branch March 26, 2015 20:41
JulienItard added a commit to JulienItard/symfony-installer that referenced this pull request Mar 26, 2015
I forget a sudo to have exactly same step symfony#122. My bad.
@JulienItard JulienItard mentioned this pull request Mar 26, 2015
javiereguiluz added a commit that referenced this pull request Mar 26, 2015
This PR was merged into the 1.0-dev branch.

Discussion
----------

Update README.md

I forget a sudo to have exactly same step #122. My bad.

Commits
-------

5c2300a Update README.md
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants