friendica/friendica-docker-ci
6
0
Fork 3
Code Issues Pull requests Projects Releases Packages Wiki Activity

Use updated PHP-CS-Fixer in standard location #16

Merged
MrPetovan merged 2 commits from toddy/friendica-docker-ci:php-cs-fixer into stable 2024-01-15 15:38:37 +01:00
Conversation 2 Commits 2 Files changed 1 +1 -1
1 changed files with 1 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files
Showing only changes of commit 733743acca - Show all commits

2
php-cs/check-php-cs.sh
View file

@ -8,4 +8,4 @@ else
EXTRA_ARGS='';
fi
./bin/dev/php-cs-fixer/vendor/bin/php-cs-fixer fix --config=.php_cs.dist -v --diff --diff-format=udiff --dry-run --using-cache=no ${EXTRA_ARGS}
./vendor/bin/php-cs-fixer check -v --diff --using-cache=no ${EXTRA_ARGS}
MrPetovan commented 2024-01-15 14:44:13 +01:00
Outdated
Review
Copy link

The previous call used the --dry-run flag, I’m not sure we should remove it.

The previous call used the `--dry-run` flag, I’m not sure we should remove it.
toddy commented 2024-01-15 14:53:03 +01:00
Outdated
Review
Copy link

I've changed the command from fix --dry-run to check, so I think it's safe to omit the --dry-run flag. If you prefer, I can add it again.

I've changed the command from `fix --dry-run` to `check`, so I think it's safe to omit the --dry-run flag. If you prefer, I can add it again.
toddy commented 2024-01-15 15:11:46 +01:00
Outdated
Review
Copy link

I've now adopted the suggestion from the php-cs-fixer documentation for running it on CI.

I've now adopted the suggestion from the php-cs-fixer documentation for running it on CI.