TYPO3 / Fluid

Fluid template rendering engine - Standalone version
GNU Lesser General Public License v3.0
151 stars 93 forks source link

[TASK] Avoid composer 'scripts' and 'scripts-descriptions' #884

Closed lolli42 closed 2 months ago

lolli42 commented 2 months ago

Fluid standalone devs should rely on the calls performed by GH workflows. It is a burden to keep composer scripts in sync.

mbrodala commented 2 months ago

Why not Zoidb^ both? Actually it's a good pattern to have the CI call commands exactly like a human developer would do. This guarantees the same results. So instead of dropping the Composer commands the Github workflows should actually run these commands. If parallel jobs are not a must, you can even have a single composer build command run by the CI.

Example:

lolli42 commented 2 months ago

Sorry, but no.

In general, I'm not a fan of composer scripts in the first place: They start with naming issues you then have to remember, gives additional headaches with additional arguments, is opinionated, and need maintenance.

In this very case, the 'dev' tasks are really simple:

composer update
vendor/bin/php-cs-fixer fix
vendor/bin/phpstan analyze
vendor/bin/phpunit

Plus the optional casual command options these binaries provide, if a dev wants that. Maybe also with a different php executor like this: /usr/bin/php8.3 vendor/bin/phpunit.

For this library, I see neither the need to encapsulate (and maintain) these binary calls every dev should know in composer.json, nor to abstract this away via something like runTests.sh - I think it's fair to require a local PHP in fluid standalone for devs who wish to develop things.