[SPIP Zone] [Spip-zone-commit] r98508 - _plugins_/coordonnees/trunk

Héhé,
tu crois que je te signale un défaut de commit pour un and en AND ? nannnna
c'est tout le fichier qui a été impacté si tu regardes bien …

Le 21/06/16 à 16:17, arnaud.berard@mister-graphx.com a écrit :

Author: arnaud.berard@mister-graphx.com
Date: 2016-06-21 16:17:11 +0200 (Tue, 21 Jun 2016)
New Revision: 98508

Modified:
    _plugins_/coordonnees/trunk/coordonnees_pipelines.php
Log:
pas de z pour ca tout de meme ! mea culpa

Details: Connexion · GitLab

_______________________________________________
Spip-zone-commit@rezo.net - http://listes.rezo.net/mailman/listinfo/spip-zone-commit

Le 21/06/2016 à 16:47, touti a écrit :

Héhé,
tu crois que je te signale un défaut de commit pour un and en AND ? nannnna
c'est tout le fichier qui a été impacté si tu regardes bien …

Le 21/06/16 à 16:17, arnaud.berard@mister-graphx.com a écrit :

Author: arnaud.berard@mister-graphx.com
Date: 2016-06-21 16:17:11 +0200 (Tue, 21 Jun 2016)
New Revision: 98508

Modified:
    _plugins_/coordonnees/trunk/coordonnees_pipelines.php
Log:
pas de z pour ca tout de meme ! mea culpa

Details: Connexion · GitLab

_______________________________________________
Spip-zone-commit@rezo.net -
http://listes.rezo.net/mailman/listinfo/spip-zone-commit

----
spip-zone@rezo.net - http://listes.rezo.net/mailman/listinfo/spip-zone

ha oui, y'a les espaces qui ont sauté, je vois que d'autant plus cerdic sur le commit suivant n'as pas respecté les mêmes rêgles de codage si je ne m'abuse., que celles mises en place par marcimat avec php-cs…

donc je vais surtout faire un autre mail et demander qu'ont se mette tous d'accord sur un standard ou un fichier de config type pour php-code-sniffer, ce sera beaucoup plus productif que de corriger les espaces a la mano, je pense.

Je te remercie de ton attention.
Bonne journée
Arnaud B. (Mist. GraphX)

Le 21/06/16 à 18:24, Mist. GraphX a écrit :

ha oui, y'a les espaces qui ont sauté, je vois que d'autant plus cerdic sur le commit suivant n'as pas respecté les mêmes rêgles de codage si je ne m'abuse., que celles mises en place par marcimat avec php-cs…

donc je vais surtout faire un autre mail et demander qu'ont se mette tous d'accord sur un standard ou un fichier de config type pour php-code-sniffer, ce sera beaucoup plus productif que de corriger les espaces a la mano, je pense.

Je te remercie de ton attention.
Bonne journée
Arnaud B. (Mist. GraphX)

Ah, et aussi avec r98508 tu as remis une fonction inutile qui avait été retirée :

++
touti

Le 21/06/2016 à 19:56, touti a écrit :

Le 21/06/16 à 18:24, Mist. GraphX a écrit :

ha oui, y'a les espaces qui ont sauté, je vois que d'autant plus
cerdic sur le commit suivant n'as pas respecté les mêmes rêgles de
codage si je ne m'abuse., que celles mises en place par marcimat avec
php-cs…

donc je vais surtout faire un autre mail et demander qu'ont se mette
tous d'accord sur un standard ou un fichier de config type pour
php-code-sniffer, ce sera beaucoup plus productif que de corriger les
espaces a la mano, je pense.

Je te remercie de ton attention.
Bonne journée
Arnaud B. (Mist. GraphX)

Ah, et aussi avec r98508 tu as remis une fonction inutile qui avait été
retirée :
Connexion · GitLab

++
touti
----
spip-zone@rezo.net - http://listes.rezo.net/mailman/listinfo/spip-zone

bon c'est remis en forme avec les rêgles de codages appliquées via le ruleset.xml de la communauté.

en espérant que ça te conviendra

Bonne journée

Super,
je ne faisais que pointer une erreur que chacun·e peut faire,
ce n'est pas une affaire perso !)
++
touti

bon c'est remis en forme avec les rêgles de codages appliquées via le ruleset.xml de la communauté.

en espérant que ça te conviendra

Bonne journée

----
spip-zone@rezo.net - http://listes.rezo.net/mailman/listinfo/spip-zone

Je t'en prie, je ne l'ai pas pris comme ça, je te soumettais juste la correction, pour que tu me donne ton avis.

il va de soit que quand on travaille ou contribue à plusieurs, on s'entende sur les procédures,

et qu'on ecrase pas le travail des autres :wink:

merci du signalement en tout cas - ça m'as de plus permis de me tester phpcs, donc tout benef :wink:

Bonne journée
Arnaud B. (Mist. GraphX)

Le 22/06/2016 à 12:09, touti a écrit :

Super,
je ne faisais que pointer une erreur que chacun·e peut faire,
ce n'est pas une affaire perso !)
++
touti

bon c'est remis en forme avec les rêgles de codages appliquées via le ruleset.xml de la communauté.

en espérant que ça te conviendra

Bonne journée

----
spip-zone@rezo.net - http://listes.rezo.net/mailman/listinfo/spip-zone