Skip to content

Fix coding standards #2644

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

Fix coding standards #2644

wants to merge 1 commit into from

Conversation

gl3n
Copy link
Contributor

@gl3n gl3n commented May 17, 2013

No description provided.

@@ -49,11 +49,11 @@ example containing most features described below:

/**
* @param string $dummy Some argument description
* @param array $options
* @param array $options
Copy link
Member

Choose a reason for hiding this comment

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

I'm not sure about this change. In the Symfony coding standards (http://symfony.com/doc/current/contributing/code/standards.html), the example shown does not include lining up the parameters. I'm also not sure if this is addressed specifically in PSR1 or PSR2. Is there a standard somewhere for this?

Thanks!

Copy link
Member

Choose a reason for hiding this comment

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

First of all, we are talking about the example in contributing/code/standards, so of course does the current example not show that :)

And to answer you question: It's done in all comments in the source code. It's also part of the, never finished, Symfony2 PHPdoc standards: symfony/symfony#4183

Copy link
Member

Choose a reason for hiding this comment

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

Ha, of course - that's really funny :). Thanks @wouterj!

@weaverryan
Copy link
Member

Patched into the 2.1 branch at sha: ab1a1fa

Thanks!

@weaverryan weaverryan closed this May 27, 2013
@gl3n
Copy link
Contributor Author

gl3n commented May 30, 2013

@weaverryan can you patch all 2.* branches or do I need to submit PR on all branches ?

@wouterj
Copy link
Member

wouterj commented May 30, 2013

It'll be fine. That's done once in a week or something like that

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.

3 participants