This PR was squashed before being merged into the 2.7 branch (closes #25653).
Discussion
----------
PHP CS Fixer: clean up repo and adjust config
| Q | A
| ------------- | ---
| Branch? | 2.7
| Bug fix? | no
| New feature? | no
| BC breaks? | no
| Deprecations? | no
| Tests pass? | n/a
| Fixed tickets | n/a
| License | MIT
| Doc PR | n/a
Reason for this PR is that one want to have `php-cs-fixer fix -v` command executed without changes that shall not be applied for this repo. To achieve that, we need to groom config to exclude files that violate CS willingly, fix files that are violating CS unwillingly, and deliver missing case handling at PHP CS Fixer itself (https://github.com/FriendsOfPHP/PHP-CS-Fixer/pull/3359) (already merged!).
Commits
-------
b14cbc1
PHP CS Fixer: clean up repo and adjust config
46 lines
2.2 KiB
PHP
46 lines
2.2 KiB
PHP
<?php
|
|
|
|
if (!file_exists(__DIR__.'/src')) {
|
|
exit(0);
|
|
}
|
|
|
|
return PhpCsFixer\Config::create()
|
|
->setRules(array(
|
|
'@Symfony' => true,
|
|
'@Symfony:risky' => true,
|
|
'@PHPUnit48Migration:risky' => true,
|
|
'php_unit_no_expectation_annotation' => false, // part of `PHPUnitXYMigration:risky` ruleset, to be enabled when PHPUnit 4.x support will be dropped, as we don't want to rewrite exceptions handling twice
|
|
'array_syntax' => array('syntax' => 'long'),
|
|
'protected_to_private' => false,
|
|
// rule disabled due to https://bugs.php.net/bug.php?id=60573 bug;
|
|
// to be re-enabled (by dropping next line, rule is part of @Symfony already) on branch that requires PHP 5.4+
|
|
'self_accessor' => false,
|
|
))
|
|
->setRiskyAllowed(true)
|
|
->setFinder(
|
|
PhpCsFixer\Finder::create()
|
|
->in(__DIR__.'/src')
|
|
->append(array(__FILE__))
|
|
->exclude(array(
|
|
// directories containing files with content that is autogenerated by `var_export`, which breaks CS in output code
|
|
'Symfony/Component/DependencyInjection/Tests/Fixtures',
|
|
'Symfony/Component/Routing/Tests/Fixtures/dumper',
|
|
// fixture templates
|
|
'Symfony/Component/Templating/Tests/Fixtures/templates',
|
|
'Symfony/Bundle/FrameworkBundle/Tests/Templating/Helper/Resources/Custom',
|
|
// resource templates
|
|
'Symfony/Bundle/FrameworkBundle/Resources/views/Form',
|
|
))
|
|
// file content autogenerated by `var_export`
|
|
->notPath('Symfony/Component/Translation/Tests/fixtures/resources.php')
|
|
// test template
|
|
->notPath('Symfony/Bundle/FrameworkBundle/Tests/Templating/Helper/Resources/Custom/_name_entry_label.html.php')
|
|
// explicit heredoc test
|
|
->notPath('Symfony/Bundle/FrameworkBundle/Tests/Fixtures/Resources/views/translation.html.php')
|
|
// explicit trigger_error tests
|
|
->notPath('Symfony/Bridge/PhpUnit/Tests/DeprecationErrorHandler/default.phpt')
|
|
->notPath('Symfony/Bridge/PhpUnit/Tests/DeprecationErrorHandler/weak.phpt')
|
|
->notPath('Symfony/Component/Debug/Tests/DebugClassLoaderTest.php')
|
|
)
|
|
;
|