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

Ref #3903 - Normalize methods listings #4550

Closed
wants to merge 2 commits into from

Conversation

ternel
Copy link
Contributor

@ternel ternel commented Nov 29, 2014

Q A
Doc fix? yes
New docs? no
Applies to 2.3+
Fixed tickets #3903

I only did this on methods listings which have some informations about the methods. (ie. I didn't change anything on this page: http://symfony.com/doc/current/contributing/code/conventions.html#method-names)

@javiereguiluz
Copy link
Member

@wouterj what do you think about this PR considering that there are other related PR about normalizing method listings?

@xabbuh
Copy link
Member

xabbuh commented Dec 18, 2014

Most of these changes are already included in #4539.

@pedronofuentes Can you maybe update your pull request to include the missing changes from this one?

@pedronofuentes
Copy link
Contributor

Ok

@xabbuh
Copy link
Member

xabbuh commented Dec 18, 2014

@pedronofuentes Thank you and sorry for the additional work.

@xabbuh
Copy link
Member

xabbuh commented Dec 21, 2014

@pedronofuentes Did you manage to include the missing changes into #4539 or should we do a rebase here to get the rest merged?

@pedronofuentes
Copy link
Contributor

@xabbuh I'm sorry, I couldn't. Can I help somehow?

@xabbuh
Copy link
Member

xabbuh commented Dec 21, 2014

@pedronofuentes Don't worry. :) We can still merge this one.

@ternel Are you able to rebase your pull request?

@ternel ternel force-pushed the normalize_method_listings branch 2 times, most recently from f88d8af to 9ee8a1f Compare December 21, 2014 12:02
@ternel
Copy link
Contributor Author

ternel commented Dec 21, 2014

It should be ok @xabbuh :)

@@ -132,6 +132,7 @@ an array. A few methods exist for "Bag" management:
Session Metadata
................


Copy link
Member

Choose a reason for hiding this comment

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

Can you reduce them to one blank line?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed

@ternel ternel force-pushed the normalize_method_listings branch from 9ee8a1f to 0912b19 Compare December 21, 2014 13:16
@xabbuh
Copy link
Member

xabbuh commented Dec 22, 2014

@ternel Thank you! Looks good. 👍

@ternel
Copy link
Contributor Author

ternel commented Dec 22, 2014

Thx for your review :)

@wouterj
Copy link
Member

wouterj commented Dec 22, 2014

It's really great to see many people working together to fix such a broad issue! Thanks for rebasing, Benjamin. You made my life a lot easier

wouterj added a commit that referenced this pull request Dec 22, 2014
This PR was squashed before being merged into the 2.3 branch (closes #4550).

Discussion
----------

Ref #3903 - Normalize methods listings

| Q             | A
| ------------- | ---
| Doc fix?      | yes
| New docs?     | no
| Applies to    | 2.3+
| Fixed tickets | #3903

I only did this on methods listings which have some informations about the methods. (ie. I didn't change anything on this page: http://symfony.com/doc/current/contributing/code/conventions.html#method-names)

Commits
-------

9039e5b Ref #3903 - Normalize methods listings
@wouterj wouterj closed this Dec 22, 2014
@ternel ternel deleted the normalize_method_listings branch December 22, 2014 11:12
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.

5 participants