DevKit updates

Author: SonataCI

.php_cs

@@ -1,40 +1,5 @@
 <?php
 
-/*
- * DO NOT EDIT THIS FILE!
- *
- * It's auto-generated by sonata-project/dev-kit package.
- *
- * Package `sllh/php-cs-fixer-styleci-bridge` is required to get it working.
- */
+// TO BE REMOVED.
 
-require_once __DIR__.'/vendor/sllh/php-cs-fixer-styleci-bridge/autoload.php';
-
-use SLLH\StyleCIBridge\ConfigBridge;
-
-$header = <<<EOF
-This file is part of the Sonata Project package.
-
-(c) Thomas Rabaix <[email protected]>
-
-For the full copyright and license information, please view the LICENSE
-file that was distributed with this source code.
-EOF;
-
-// PHP-CS-Fixer 1.x
-if (class_exists('Symfony\CS\Fixer\Contrib\HeaderCommentFixer')) {
-    \Symfony\CS\Fixer\Contrib\HeaderCommentFixer::setHeader($header);
-}
-
-$config = ConfigBridge::create()
-    ->setUsingCache(true)
-;
-
-// PHP-CS-Fixer 2.x
-if (method_exists($config, 'setRules')) {
-    $config->setRules(array_merge($config->getRules(), array(
-        'header_comment' => array('header' => $header)
-    )));
-}
-
-return $config;
+return require_once __DIR__.'/.php_cs.dist';

.php_cs.dist

@@ -0,0 +1,76 @@
+<?php
+
+/**
+ * DO NOT EDIT THIS FILE!
+ *
+ * It's auto-generated by sonata-project/dev-kit package.
+ */
+
+$header = <<<'HEADER'
+This file is part of the Sonata Project package.
+
+(c) Thomas Rabaix <[email protected]>
+
+For the full copyright and license information, please view the LICENSE
+file that was distributed with this source code.
+HEADER;
+
+$rules = [
+    '@Symfony' => true,
+    '@Symfony:risky' => true,
+    'array_syntax' => [
+        'syntax' => 'short',
+    ],
+    'combine_consecutive_issets' => true,
+    'combine_consecutive_unsets' => true,
+    'header_comment' => [
+        'header' => $header,
+    ],
+    'no_extra_consecutive_blank_lines' => true,
+    'no_php4_constructor' => true,
+    'no_useless_else' => true,
+    'no_useless_return' => true,
+    'ordered_class_elements' => true,
+    'ordered_imports' => true,
+    'phpdoc_order' => true,
+    'ternary_to_null_coalescing' => true,
+    // To be tested before insertion:
+//    'strict_comparison' => true,
+//    'strict_param' => true,
+//    'php_unit_strict' => true,
+];
+
+if (\PHP_VERSION_ID >= 50600) {
+    $rules = array_merge($rules, [
+        '@PHP56Migration' => true,
+        '@PHP56Migration:risky' => true,
+        '@PHPUnit57Migration:risky' => true,
+    ]);
+}
+if (\PHP_VERSION_ID >= 70000) {
+    $rules = array_merge($rules, [
+        '@PHP70Migration:risky' => true,
+        '@PHPUnit60Migration:risky' => true,
+    ]);
+}
+if (\PHP_VERSION_ID >= 70100) {
+    $rules = array_merge($rules, [
+        '@PHP71Migration:risky' => true,
+        'compact_nullable_typehint' => true,
+    ]);
+}
+
+$finder = PhpCsFixer\Finder::create()
+    ->in(preg_match('/dev-kit\/project$/', __DIR__, $matches) ? __DIR__.'/..' : __DIR__)
+    ->exclude('Tests/Fixtures')
+    ->exclude('tests/Fixtures')
+    ->exclude('Resources/skeleton')
+    ->exclude('Resources/public/vendor')
+;
+
+return PhpCsFixer\Config::create()
+    ->setFinder($finder)
+    ->setRiskyAllowed(true)
+    ->setRules($rules)
+    ->setUsingCache(true)
+;

.styleci.yml

@@ -1,30 +1 @@
-# DO NOT EDIT THIS FILE!
-#
-# It's auto-generated by sonata-project/dev-kit package.
-#
-# Package `sllh/php-cs-fixer-styleci-bridge` is required to get it working.
-
-preset: symfony
-
-enabled:
-  - combine_consecutive_unsets
-  - short_array_syntax
-  - newline_after_open_tag
-  - no_php4_constructor
-  - no_useless_else
-  - ordered_class_elements
-  - ordered_use
-# Comment strict rules for the moment. Should be uncomment later to see StyleCI PR results
-#  - strict
-#  - strict_param
-#  - php_unit_construct
-#  - php_unit_strict
-
-finder:
-  exclude:
-    - 'Tests/Fixtures'
-    - 'tests/Fixtures'
-    # ecommerce special case:
-    - 'Resources/skeleton'
-    # ignore vendor assets
-    - 'Resources/public/vendor'
+# TO BE REMOVED

.travis.yml

@@ -13,6 +13,7 @@ php:
   - '5.6'
   - '7.0'
   - '7.1'
+  - '7.2'
   - nightly
 
 sudo: false
@@ -34,29 +35,29 @@ env:
 matrix:
   fast_finish: true
   include:
-    - php: '7.1'
+    - php: '7.2'
       env: TARGET=docs
-    - php: '7.1'
+    - php: '5.6'
       env: TARGET=lint
     - php: '5.6'
       env: COMPOSER_FLAGS="--prefer-lowest"
-    - php: '7.1'
+    - php: '7.2'
       env: SYMFONY=2.8.*
-    - php: '7.1'
+    - php: '7.2'
       env: SYMFONY='dev-master as 2.8.x-dev'
-    - php: '7.1'
+    - php: '7.2'
       env: FOS_USER=1.3.*
-    - php: '7.1'
+    - php: '7.2'
       env: FOS_USER='dev-master as 1.3.x-dev'
-    - php: '7.1'
+    - php: '7.2'
       env: SONATA_CORE=3.*
-    - php: '7.1'
+    - php: '7.2'
       env: SONATA_CORE='dev-master as 3.x-dev'
-    - php: '7.1'
+    - php: '7.2'
       env: SONATA_ADMIN=3.*
-    - php: '7.1'
+    - php: '7.2'
       env: SONATA_ADMIN='dev-master as 3.x-dev'
-    - php: '7.1'
+    - php: '7.2'
       env: SYMFONY_DEPRECATIONS_HELPER=0
   allow_failures:
     - php: nightly

.travis/install_lint.sh

@@ -1,6 +1,11 @@
 #!/usr/bin/env sh
 set -ev
 
+mkdir --parents "${HOME}/bin"
+
+wget "http://cs.sensiolabs.org/download/php-cs-fixer-v2.phar" --output-document="${HOME}/bin/php-cs-fixer"
+chmod u+x "${HOME}/bin/php-cs-fixer"
+
 composer global require sllh/composer-lint:@stable --prefer-dist --no-interaction
 
 gem install yaml-lint

CONTRIBUTING.md

@@ -72,6 +72,23 @@ and run this command before committing your modifications:
 php-cs-fixer fix --verbose
 ```
 
+Please note that we try to keep phpdoc to a minimum, so if an `@param` phpdoc
+comment brings nothing more than the type hint and variable name already do,
+it should be removed. Descriptions are optional if you want to document a type.
+
+```php
+/**
+ * @param Bar|Baz $foo
+ * @param int     $limit a crucial, highly interesting comment
+ */
+protected function bar($foo, string $name, int $limit)
+{
+    // ...
+}
+```
+
+Please also note that multiline signatures are allowed when the line is longer than 120 characters.
+
 #### The documentation
 
 The documentation is mostly written with the `rst` format, and can be found in the `docs` directory.

Makefile

@@ -13,6 +13,7 @@ lint:
 	find . \( -name '*.xml' -or -name '*.xliff' \) \
 		-not -path './vendor/*' -not -path './Resources/public/vendor/*' -type f \
 		-exec xmllint --encode UTF-8 --output '{}' --format '{}' \;
+	php-cs-fixer fix --verbose
 	git diff --exit-code
 
 test: