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

added composer info #5491

Closed
wants to merge 1 commit into from
Closed

Conversation

OskarStark
Copy link
Contributor

No description provided.

@@ -121,6 +121,13 @@ To use this component, add it as a dependency of the project:

Running this command will also automatically download the Symfony
HttpFoundation component and install it under the ``vendor/`` directory.
A ``composer.json`` and a ``composer.lock`` file will be generated as well.::
Copy link
Member

Choose a reason for hiding this comment

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

The trailing dot after well should be removed.

Copy link
Member

Choose a reason for hiding this comment

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

The double :: "hack" can only be used when the code to highlight is PHP. In this case we need to highlight JSON, so you'll have to use a single :: and add a .. code-block:: directive.

@OskarStark
Copy link
Contributor Author

@javiereguiluz done, can you please check?


.. code-block:: json

// composer.json
Copy link
Member

Choose a reason for hiding this comment

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

JSON does not have comments. So I think we should be avoid such constructs here.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

shall ii use .. code-block:: text instead?

Copy link
Member

Choose a reason for hiding this comment

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

Well, I don't consider this as an issue with the syntax highlighting. It's just that someone will probably copy the code example and will then run into syntax errors because the example itself is no valid JSON.

I would rather change the example and maybe write an explanation below the code.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done @xabbuh

"symfony/http-foundation": "^2.7"
}

``The code block shows the content of the ``composer.json`` file (the actual
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@xabbuh you mean the backticks here?

Copy link
Member

Choose a reason for hiding this comment

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

@OskarStark Yeah, but they should only enclose the file name. So you should remove them from the beginning and the end of the paragraph now. :)

@OskarStark
Copy link
Contributor Author

done @xabbuh

@xabbuh
Copy link
Member

xabbuh commented Jul 10, 2015

cool 👍 looks ready to be merged now

wouterj added a commit that referenced this pull request Jul 23, 2015
This PR was submitted for the 2.7 branch but it was merged into the 2.3 branch instead (closes #5491).

Discussion
----------

added composer info

Commits
-------

944a795 added composer info
wouterj added a commit that referenced this pull request Jul 23, 2015
@wouterj
Copy link
Member

wouterj commented Jul 23, 2015

👍! Merged it with a few minor changes (mostly fixing the syntax error in the code example) in ae14002

@OskarStark
Copy link
Contributor Author

Thank you @wouterj

@OskarStark OskarStark deleted the composer-info branch June 17, 2019 06:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants