Skip to content

[Reference][Requirements] Fix missing literal formatting #3155

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

Merged
merged 3 commits into from
Nov 5, 2013
Merged

[Reference][Requirements] Fix missing literal formatting #3155

merged 3 commits into from
Nov 5, 2013

Conversation

bicpi
Copy link
Contributor

@bicpi bicpi commented Nov 5, 2013

Q A
Doc fix? yes
New docs? no
Applies to 2.2+
Fixed tickets -

@@ -22,7 +22,7 @@ Required
* PHP needs to be a minimum version of PHP 5.3.3
* JSON needs to be enabled
* ctype needs to be enabled
* Your PHP.ini needs to have the date.timezone setting
* Your PHP.ini needs to have the ``date.timezone`` setting
Copy link
Member

Choose a reason for hiding this comment

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

PHP.ini can also be changed to php.ini

@bicpi
Copy link
Contributor Author

bicpi commented Nov 5, 2013

@xabbuh Thanks, I was also thinking about this and I have updated this in every place I found from PHP ini and PHP.ini to php.ini. Now this branch has a misleading name ...

@wouterj
Copy link
Member

wouterj commented Nov 5, 2013

php.ini is a filename, so it should be placed inside a literal

@bicpi
Copy link
Contributor Author

bicpi commented Nov 5, 2013

@wouterj Thanks, I've added the literal format

weaverryan added a commit that referenced this pull request Nov 5, 2013
[Reference][Requirements] Fix missing literal formatting
@weaverryan weaverryan merged commit 7d1f954 into symfony:2.2 Nov 5, 2013
@bicpi bicpi deleted the fix_missing_literal_formatting branch January 5, 2014 13:24
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