From c2f39b4034d34f493b9a53d033bbaf193b7ceafa Mon Sep 17 00:00:00 2001 From: Wouter J Date: Mon, 25 May 2015 09:43:34 +0200 Subject: [PATCH] Some fixes for bundle best practices --- cookbook/bundles/best_practices.rst | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/cookbook/bundles/best_practices.rst b/cookbook/bundles/best_practices.rst index 02622aab9fd..58f01c74366 100644 --- a/cookbook/bundles/best_practices.rst +++ b/cookbook/bundles/best_practices.rst @@ -146,7 +146,7 @@ class name is ``Acme\BlogBundle\Controller\ContentController``. All classes and files must follow the :doc:`Symfony coding standards `. Some classes should be seen as facades and should be as short as possible, like -Commands, Helpers, Listeners, and Controllers. +Commands, Helpers, Listeners and Controllers. Classes that connect to the event dispatcher should be suffixed with ``Listener``. @@ -159,7 +159,7 @@ Vendors A bundle must not embed third-party PHP libraries. It should rely on the standard Symfony autoloading instead. -A bundle should not embed third-party libraries written in JavaScript, CSS, or +A bundle should not embed third-party libraries written in JavaScript, CSS or any other language. Tests @@ -175,6 +175,7 @@ the ``Tests/`` directory. Tests should follow the following principles: * The tests should cover at least 95% of the code base. .. note:: + A test suite must not contain ``AllTests.php`` scripts, but must rely on the existence of a ``phpunit.xml.dist`` file.