le-phare / flex-recipes

Le Phare Recipes Repository
https://flex.lephare.io
MIT License
2 stars 1 forks source link

fix(php-cs-fixer): separate from code-quality-pack, add new rule sets, change cache path #19

Open lucasmirloup opened 1 year ago

lucasmirloup commented 1 year ago

Closes #14

As of v3.11:

lucasmirloup commented 1 year ago

@thislg Y'a plusieurs modifs dans la PR, certaines sont débattables :wink:

Je pars du principe que cette recette ne va s'appliquer qu'aux nouveaux projets ou à ceux ayant PHP CS Fixer >= 3.11.

Du coup je pense qu'on peut se permettre d'enfin utiliser PHP CS Fixer à son plein potentiel. Ex: les rule sets risky (qui ne portent pas très bien leur nom :smile:) : en regardant de près y'a rien de vraiment dangereux. Ils sont séparés des rule sets normaux pour que les gens ne cassent pas les pieds aux devs de la lib dans des cas pourris type "j'ai surchargé les fonctions natives PHP dans mes scripts".

Sur le sujet de la méthode de recette Flex, je ne sais toujours pas si ça serait mieux de garder ce fichier de config dans le code-quality-pack et de déclarer un conflit avec la recette Flex officielle CS-Fixer :shrug:.