codings standards SCS1 #28
Merged
b_b
merged 4 commits from coding_standards into master 6 months ago
Loading…
Reference in new issue
There is no content yet.
Delete Branch 'coding_standards'
Deleting a branch is permanent. It CANNOT be undone. Continue?
une passe de phpcbf avec exclusion du sniff Generic.Arrays.DisallowLongArraySyntax et sans toucher au fichier inc/iptc.php
pour référence, la commande utilisée
vendor/bin/phpcbf plugins/gis --exclude=Generic.Arrays.DisallowLongArraySyntax --ignore=lang/*,inc/iptc.phpPour info, j'hésite à appliquer Generic.Arrays.DisallowLongArraySyntax pour convertir les
array()en[](syntaxe dispo à partir de PHP 5.4) car le plugin est marqué compatible à partir de SPIP 3.0 alors que cette version est compatible PHP 5.1 cf https://www.spip.net/fr_article4351.htmlAprès, la 3.0 et la 3.1 ne sont plus maintenues, mais autant être sympa avec les gens qui l'utilisent encore. Sinon, je change la version de compat pour la remonter à 3.2 mini et zou, les vieux sites resteront sur le dernier tag en date de GIS. Vos avis ?
Plan B, ajouter un necessite PHP 5.4 dans le paquet.
@marcimat @rastapopoulos votre avis ?
je pense qu'en 2022, on peut imposer si l'on souhaite faire une maj de son site avoir à minima php 5.4. Donc +1 pour l'ajout du necessite PHP 5.4
Ouais quand même là, c'est même pas PHP 7.4… mais 5.4, c'est tellement vieux qu'on peut l'imposer oui
Et voilà, je pense que c'est bon pour le merge :)
f943e1dbdfinto master 6 months agof943e1dbdf.