codings standards SCS1 #28

Merged
b_b merged 4 commits from coding_standards into master 6 months ago
b_b commented 7 months ago
Collaborator

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.php

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.php`
b_b added 1 commit 7 months ago
9cc447da15 codings standards SCS1
Poster
Collaborator

Pour 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.html

Aprè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 ?

Pour 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.html Aprè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 ?
Poster
Collaborator

Plan B, ajouter un necessite PHP 5.4 dans le paquet.

@marcimat @rastapopoulos votre avis ?

Plan B, ajouter un necessite PHP 5.4 dans le paquet. @marcimat @rastapopoulos votre avis ?
Collaborator

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

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

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
b_b added 2 commits 6 months ago
Poster
Collaborator

Et voilà, je pense que c'est bon pour le merge :)

Et voilà, je pense que c'est bon pour le merge :)
b_b added 1 commit 6 months ago
b_b merged commit f943e1dbdf into master 6 months ago
b_b deleted branch coding_standards 6 months ago
b_b referenced this issue from a commit 4 months ago
The pull request has been merged as f943e1dbdf.
Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
3 Participants
Notifications
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Loading…
There is no content yet.