Skip to content

Move annotations example to front #4893

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 1 commit into from
Jan 30, 2015
Merged

Move annotations example to front #4893

merged 1 commit into from
Jan 30, 2015

Conversation

ifdattic
Copy link
Contributor

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

| Q             | A
| ------------- | ---
| Doc fix?      | yes
| New docs?     | no
| Applies to    | 2.3
| Fixed tickets |
@stof
Copy link
Member

stof commented Jan 21, 2015

it should be done in all files, not only in 1

@ifdattic
Copy link
Contributor Author

As this is the best practice I think most of them have annotations moved to the front, this is just one of the leftover.

@weaverryan
Copy link
Member

Thanks @ifdattic!

@weaverryan weaverryan merged commit aaecb02 into symfony:2.3 Jan 30, 2015
weaverryan added a commit that referenced this pull request Jan 30, 2015
This PR was merged into the 2.3 branch.

Discussion
----------

Move annotations example to front

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

Commits
-------

aaecb02 Move annotations example to front
@weaverryan
Copy link
Member

Oh, and I think there are more that don't show annotations or don't show them first, but we will re-work those little by little.

@ifdattic ifdattic deleted the patch-12 branch January 30, 2015 09:14
@ifdattic
Copy link
Contributor Author

So far I found only two of them. Still have about half of components section to read and some other misc pages if I will find any of them will send a PR

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.

4 participants