Skip to content

Update conventions.rst #4542

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 1 commit into from
Closed

Update conventions.rst #4542

wants to merge 1 commit into from

Conversation

csuarez
Copy link
Contributor

@csuarez csuarez commented Nov 29, 2014

The trigger_error call seems to be written in one line: https://github.com/symfony/symfony/pull/12671/files

The ``trigger_error``call seems to be written in one line:  https://github.com/symfony/symfony/pull/12671/files
@xabbuh
Copy link
Member

xabbuh commented Nov 29, 2014

It's indeed a good idea to make this change even if the readability is a bit limited. But the core team prefers to have this in one line. So people can then simply copy and paste from the docs.

@wouterj
Copy link
Member

wouterj commented Nov 29, 2014

Let's not wait on travis (as the hackday produces so many jobs, many are in the queue...) and merge this straight away. Thanks @csuarez, this will definitely help people the comming 2 hours!

wouterj added a commit that referenced this pull request Nov 29, 2014
This PR was submitted for the 2.6 branch but it was merged into the 2.3 branch instead (closes #4542).

Discussion
----------

Update conventions.rst

The ``trigger_error`` call seems to be written in one line:  https://github.com/symfony/symfony/pull/12671/files

Commits
-------

3d75f6a Update conventions.rst
@wouterj wouterj closed this Nov 29, 2014
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.

3 participants