Skip to content

Commit 722f8c8

Browse files
committed
fix php-cs-fixer
1 parent 6855c7c commit 722f8c8

File tree

4 files changed

+24
-5
lines changed

4 files changed

+24
-5
lines changed

.php_cs.dist renamed to .php_cs.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,4 +26,4 @@
2626
PhpCsFixer\Finder::create()
2727
->in(__DIR__)
2828
)
29-
;
29+
;

.travis.yml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -53,9 +53,9 @@ install:
5353
- if [ "$PREPARE_CONTAINER" = true ]; then bin/dev -b; fi
5454

5555
script:
56-
- IFS=$'\n'; COMMIT_SCA_FILES=($(git diff --name-only --diff-filter=ACMRTUXB "${TRAVIS_COMMIT_RANGE}")); unset IFS
57-
- if [ "$PHP_CS_FIXER" = true ]; then ./bin/php-cs-fixer --no-interaction --dry-run --diff -v --path-mode=intersection fix -- "${COMMIT_SCA_FILES[@]}" ; fi
58-
- if [ "$PHPSTAN" = true ]; then docker run --workdir="/mqdev" -v "`pwd`:/mqdev" --rm enqueue/dev:latest php -d memory_limit=1024M bin/phpstan analyse -l 1 -c phpstan.neon -- "${COMMIT_SCA_FILES[@]}" ; fi
56+
- PKG_PHP_CHANGED_FILES=`./bin/git-find-changed-php-files.sh "${TRAVIS_COMMIT_RANGE}"`
57+
- if [ "$PHP_CS_FIXER" = true ] && [ ! -z "${PKG_PHP_CHANGED_FILES}" ]; then ./bin/php-cs-fixer fix --config=.php_cs.php --no-interaction --dry-run --diff -v --path-mode=intersection -- ${PKG_PHP_CHANGED_FILES[@]} ; fi
58+
- if [ "$PHPSTAN" = true ] && [ ! -z "${PKG_PHP_CHANGED_FILES}" ]; then docker run --workdir="/mqdev" -v "`pwd`:/mqdev" --rm enqueue/dev:latest php -d memory_limit=1024M bin/phpstan analyse -l 1 -c phpstan.neon -- ${PKG_PHP_CHANGED_FILES[@]} ; fi
5959
- if [ "$UNIT_TESTS" = true ]; then bin/phpunit --exclude-group=functional; fi
6060
- if [ "$FUNCTIONAL_TESTS" = true ]; then bin/test.sh --exclude-group=rdkafka; fi
6161
- if [ "RDKAFKA_TESTS" = true ]; then bin/test.sh --group=rdkafka; fi

bin/git-find-changed-php-files.sh

Lines changed: 15 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,15 @@
1+
#!/usr/bin/env bash
2+
3+
if (( "$#" != 1 ))
4+
then
5+
echo "Git range must be provided"
6+
exit 1
7+
fi
8+
9+
10+
IFS='
11+
'
12+
ALL_CHANGED_FILES=$(git diff --name-only --diff-filter=ACMRTUXB "$1");
13+
PKG_PHP_CHANGED_FILES=$(echo "$ALL_CHANGED_FILES" | grep -E "^pkg\/" | grep -E ".*?\.php$");
14+
15+
echo "$PKG_PHP_CHANGED_FILES";

bin/pre-commit

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -68,6 +68,10 @@ function getFilesToFix()
6868
return (bool) preg_match('/\.(php|twig|translations\/*.yml)$/', $file);
6969
});
7070

71+
$stagedFiles = array_filter($stagedFiles, function ($file) {
72+
return (bool) preg_match('/^pkg\//', $file);
73+
});
74+
7175
return $stagedFiles;
7276
}
7377

@@ -104,7 +108,7 @@ function runPhpCsFixer()
104108
$returnCode = null;
105109

106110
exec(sprintf(
107-
'%s %s fix %s --dry-run',
111+
'%s %s fix %s --dry-run --config=.php_cs.php',
108112
$phpBin,
109113
$phpCsFixerBin,
110114
$projectRootDir.'/'.$file

0 commit comments

Comments
 (0)