Bienvenue, Invité
Nom d'utilisateur : Mot de passe : Se souvenir de moi
  • Page :
  • 1
  • 2

SUJET : Abandonner Travis!

Abandonner Travis! il y a 3 semaines 3 jours #114227

  • jtraulle
  • Portrait de jtraulle
  • Hors ligne
  • Admin
  • Messages : 226
  • Remerciements reçus 83
  • Karma: 15
En effet, je vais essayer de voir pour compléter la page ;)

J'aimerai également voir pour expliquer comment déployer Dolibarr en utilisant Docker pour le développement sur le Wiki. J'en profiterai pour détailler comment exécuter PHP_CodeSniffer et les tests en local (sans aller jusqu'à déployer une pile Travis-CI en local, c'est déjà d'une grande aide vu que 80% des erreurs de l'intégration continue vient du "Code style" non respecté).

C'est également possible de configurer PHPStorm par exemple pour signaler les erreurs de "Code style" en utilisant PHP_CodeSniffer et un fichier de règles comme Dolibarr en fournit un. Je vois pour documenter tout ça.
Dernière édition: il y a 3 semaines 3 jours par jtraulle.
L'administrateur a désactivé l'accès en écriture pour le public.

Abandonner Travis! il y a 3 semaines 3 jours #114229

  • dolibarr95
  • Portrait de dolibarr95
  • Hors ligne
  • Admin
  • Messages : 1270
  • Remerciements reçus 188
  • Karma: 25
Bonjour :)
Hier j'ai donc installé sur mon pc Atom avec php_codesniffer (PSR2) pour info sur un de mes fichiers pris au hasard j'ai 1684 erreurs :whistle: .
J'ai toujours fait au mieux dans mes fichiers (syntaxe, espaces, casse, etc). Ce fameux php_codesniffer me file un bon coup demain à tout corriger, à prendre les bonnes habitudes, avoir un code nickel je sais pas...mais bien moins pire oui :laugh:
Bien vu pour le wiki ;)
L'administrateur a désactivé l'accès en écriture pour le public.

Abandonner Travis! il y a 3 semaines 3 jours #114231

  • jtraulle
  • Portrait de jtraulle
  • Hors ligne
  • Admin
  • Messages : 226
  • Remerciements reçus 83
  • Karma: 15
@dolibarr95 attention, Dolibarr n'utilise pas la PSR2 direct mais un standard personnalisé. Tu peux configurer PHP_CodeSniffer en utilisant le fichier de règles fourni avec le code de Dolibarr ici : github.com/Dolibarr/dolibarr/blob/develo...esniffer/ruleset.xml

Depuis le répertoire principal :
phpcs -s -p -d memory_limit=-1 --extensions=php --colors --tab-width=4 --standard=dev/setup/codesniffer/ruleset.xml --encoding=utf-8 --runtime-set ignore_warnings_on_exit true .

Tu peux également rajouter l'option -n pour n'afficher que les erreurs et pas les avertissements ;)
L'administrateur a désactivé l'accès en écriture pour le public.
Cet utilisateur a été remercié pour son message par: dolibarr95

Abandonner Travis! il y a 3 semaines 3 jours #114233

  • dolibarr95
  • Portrait de dolibarr95
  • Hors ligne
  • Admin
  • Messages : 1270
  • Remerciements reçus 188
  • Karma: 25
Déjà que j'ai mis la journée a réussir à faire tourner tout ça correctement sous windows le personnaliser va me prendre la semaine :laugh: :laugh: :laugh: :laugh:
L'administrateur a désactivé l'accès en écriture pour le public.

Abandonner Travis! il y a 3 semaines 3 jours #114239

  • jtraulle
  • Portrait de jtraulle
  • Hors ligne
  • Admin
  • Messages : 226
  • Remerciements reçus 83
  • Karma: 15
Si tu utilises atom.io/packages/linter-phpcs tu peux paramétrer un chemin vers un fichier ruleset.xml personnalisé dans les préférences du plugin.

Mais bon, c'est vrai que Windows pour développer ... bof :whistle:
J'préfères GNU/Linux mais bon, j'suis pas très objectif :P
L'administrateur a désactivé l'accès en écriture pour le public.
Cet utilisateur a été remercié pour son message par: dolibarr95

Abandonner Travis! il y a 3 semaines 3 jours #114242

  • dolibarr95
  • Portrait de dolibarr95
  • Hors ligne
  • Admin
  • Messages : 1270
  • Remerciements reçus 188
  • Karma: 25
Oui mon pc est devenu une usine à gaz juste pour faire tourner 2,3 truc à peu près correctement :laugh:
L'administrateur a désactivé l'accès en écriture pour le public.

Abandonner Travis! il y a 3 semaines 3 jours #114247

  • pm17
  • Portrait de pm17
  • Hors ligne
  • Modérateur
  • Linux/Mac/Windows Un petit merci sur mon profil
  • Messages : 1346
  • Remerciements reçus 184
  • Karma: 23
@wdammak

Perd pas espoir
Apparament meme les "anciens" devs comme aspangaro/eldy/...
ne passe pas tjrs le travis


@jtraulle
tu sais que ton pseudo c est presque "je trolle" ? :laugh:

Bon pour travis ....
je dirais plutot une page dedié "Verifier /valider son code" ou dora explore travis :P
ca risque de rendre la page un peu lourde a lire sinon

bonne idée (comme d hab ) que tu crée une page sur docker/dolibarr
L'administrateur a désactivé l'accès en écriture pour le public.

Abandonner Travis! il y a 3 semaines 3 jours #114255

  • Arre
  • Portrait de Arre
  • Hors ligne
  • Modérateur
  • Envie de participer ? wiki -> adhérer à l'asso :)
  • Messages : 1881
  • Remerciements reçus 428
  • Karma: 44
@pm17

il faut qu'on discute sur discorde : à ta dispo dans la journée.
L'administrateur a désactivé l'accès en écriture pour le public.

Abandonner Travis! il y a 3 semaines 3 jours #114288

  • wdammak
  • Portrait de wdammak
  • Hors ligne
  • Bannis
  • Messages : 207
  • Remerciements reçus 21
  • Karma: 0
@pm17
Je perd pas espoir mais je délaisse car je ne suis plus dans le dev à 100%, juste de temps en temps question d'entrainer ma mémoire... pour cela j'utilise notepad++ (largement suffisant pour développer proprement.... sans utiliser les IDE qui généralement polluent les performances des applications...)

@jtraulle
Oui et non!
A mon avis, faut mieux se passer de Travis en mode dev et se contenter juste de Hound puis avant la publication d'une nouvelle version/branch/release faire ces tests de Travis(maquillage) sur l'ensemble!
Cela va libérer plus rapidement les PR et le projet dans son ensemble!
J'ai remarqué que des fois un PR traine deux ou 3mois et hop! les conflits se multiple ... le développeur perd espoir et abondonne...
à noter qu'il y a encore des PR très constructive qui date de 2 et 3ans, si ces PR étaient déjà libérés d'autres contributions auraient le jour...
L'administrateur a désactivé l'accès en écriture pour le public.
Cet utilisateur a été remercié pour son message par: altatof

Abandonner Travis! il y a 3 semaines 3 jours #114290

  • jtraulle
  • Portrait de jtraulle
  • Hors ligne
  • Admin
  • Messages : 226
  • Remerciements reçus 83
  • Karma: 15
@wdammak Je respecte ton point de vue mais je ne le partage pas.

Je ne voit pas en quoi un IDE pollue les performances des applications. IDE ou éditeur de texte sont simplement des outils à la disposition des développeurs qui les utilisent pour produire des logiciels. La qualité ou la performance du code en sortie dépend du développeur qui le tape, pas de l'outil qu'il utilise pour taper :laugh:

Quand à se passer de l'intégration continue pendant toute la période de développement jusqu'à la release effective d'une nouvelle version, ce n'est juste pas réalisable. Chaque développeur qui souhaite introduire des changements dans le code est "responsable" de la qualité des changements apportés et l'intégration continue permet de garantir cela selon des exigences définies collégialement. En admettant qu'on fasse comme tu dis, qui est responsable de nettoyer les écuries et corriger tout ce qui est en erreur lors de la release effective ? Le mainteneur ? Pas très gratifiant et surtout, pas bien réparti ...

Par contre, je trouve que l'intégration continue ne devrait pas retourner d'erreur sur des fichiers autres que ceux touchés par la PR en question (cela suppose que aucune PR ne soit fusionnée si l'intégration continue est en erreur et que personne ne pousse en direct dans les branches principales même la core team mais utilise des branches de feature ou fix qui seront ensuite fusionnées via PR dans la branche develop ou les branches de versions).
L'administrateur a désactivé l'accès en écriture pour le public.
  • Page :
  • 1
  • 2