Skip to content
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

6555 link to Download instructions page & Windows executable #6585

Closed
wants to merge 4 commits into from
Closed

6555 link to Download instructions page & Windows executable #6585

wants to merge 4 commits into from

Conversation

snoek09
Copy link

@snoek09 snoek09 commented May 21, 2016

Q A
Doc fix? yes
New docs? no
Applies to all
Fixed tickets #6555

I decided to not copy any instructions as mentioned on https://getcomposer.org/download/

WARNING: Please do not redistribute the install code. It will change with every version of the installer. Instead, please link to this page.

To install Composer on Linux or Mac OS X, execute the following two commands:
Run the installer following the Download page instructions from https://getcomposer.org/download.

After this you can run this command to move composer.phar to a directory that is in your path:
Copy link
Member

Choose a reason for hiding this comment

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

the filename should be enclosed by literals (double backticks)

@wouterj
Copy link
Member

wouterj commented May 21, 2016

This is a lot more future-proof than the previous article. Thanks @snoek09!


.. code-block:: bash

$ curl -sS https://getcomposer.org/installer | php
$ sudo mv composer.phar /usr/local/bin/composer
Copy link
Member

Choose a reason for hiding this comment

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

please indent this block with 3 spaces (this way, it's part of the second item of the list)


.. code-block:: bash

$ php installer

$ sudo mv composer.phar /usr/local/bin/composer
Copy link
Member

Choose a reason for hiding this comment

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

The whole code block must be indented by one space less to not be rendered as a quote.

@wouterj
Copy link
Member

wouterj commented May 21, 2016

👍

@xabbuh
Copy link
Member

xabbuh commented May 21, 2016

Thank you @snoek09.

xabbuh added a commit that referenced this pull request May 21, 2016
…ble (snoek09)

This PR was squashed before being merged into the 2.3 branch (closes #6585).

Discussion
----------

6555 link to Download instructions page & Windows executable

| Q             | A
| ------------- | ---
| Doc fix?      | yes
| New docs?     | no
| Applies to    | all
| Fixed tickets | #6555

I decided to not copy any instructions as mentioned on https://getcomposer.org/download/

> WARNING: Please do not redistribute the install code. It will change with every version of the installer. Instead, please link to this page.

Commits
-------

1967b9d 6555 link to Download instructions page & Windows executable
@xabbuh xabbuh closed this May 21, 2016
@snoek09 snoek09 deleted the 6555-update-composer-install-instructions branch May 21, 2016 13:59
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.

None yet

3 participants