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

Update cookbook entries with best practices #5674

Merged

Conversation

JhonnyL
Copy link
Contributor

@JhonnyL JhonnyL commented Sep 4, 2015

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

@javiereguiluz
Copy link
Member

A big 👍 for this change! Thanks @JhonnyL for helping us getting rid of the old Acme bundle.

@xabbuh
Copy link
Member

xabbuh commented Sep 5, 2015

👍

@xabbuh
Copy link
Member

xabbuh commented Sep 5, 2015

@JhonnyL Did you make some changes? Looks like some rebase went wrong.

@JhonnyL JhonnyL force-pushed the update-cookbook-entries-with-best-practices branch from b66a837 to 4e3ecb0 Compare September 5, 2015 09:52
@JhonnyL
Copy link
Contributor Author

JhonnyL commented Sep 5, 2015

@xabbuh Sorry, should be fixed now

@xabbuh
Copy link
Member

xabbuh commented Sep 5, 2015

Thank you @JhonnyL.

@xabbuh xabbuh merged commit 4e3ecb0 into symfony:2.3 Sep 5, 2015
xabbuh added a commit that referenced this pull request Sep 5, 2015
This PR was merged into the 2.3 branch.

Discussion
----------

Update cookbook entries with best practices

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

Commits
-------

4e3ecb0 Update cookbook entries with best practices
@@ -161,17 +161,17 @@ to and from the issue number and the ``Issue`` object::
namespace AppBundle\Form\DataTransformer;

use AppBundle\Entity\Issue;
use Doctrine\Common\Persistence\EntityManager;
use Doctrine\Common\Persistence\ObjectManager;
Copy link
Member

Choose a reason for hiding this comment

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

the changes in this document seems not to be related to this PR (but to #5666 instead, which I just merged).

@JhonnyL JhonnyL deleted the update-cookbook-entries-with-best-practices branch September 7, 2015 08:53
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