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

SUJET : Abandonner Travis!

Abandonner Travis! il y a 4 semaines 8 heures #114065

  • wdammak
  • Portrait de wdammak
  • Hors ligne
  • Bannis
  • Messages : 207
  • Remerciements reçus 21
  • Karma: 0
Il n'est pas instantané et très handicapant pour l'évolution des projets...
des fois il faut 3 PR/3jours/3semaines(si on a rien a foutre dans nos vies) pour ajouter/supprimer un espace avant/apres un "."
c'est vraiment de la merde!
Dernière édition: il y a 4 semaines 8 heures par wdammak.
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 4 semaines 8 heures #114067

  • Arre
  • Portrait de Arre
  • Hors ligne
  • Modérateur
  • Envie de participer ? wiki -> adhérer à l'asso :)
  • Messages : 1881
  • Remerciements reçus 428
  • Karma: 44
euhhh ok,
peux tu préciser ta pensée stp ? autant pour toi et dans ton attitude de "colère" : ça doit avoir du sens.

Autant pour les lecteurs de ton post : on, a aucune idée de quoi tu parles
L'administrateur a désactivé l'accès en écriture pour le public.

Abandonner Travis! il y a 4 semaines 6 heures #114068

  • aspangaro-OpenDSI
  • Portrait de aspangaro-OpenDSI
  • Hors ligne
  • Modérateur
  • Messages : 1355
  • Remerciements reçus 352
  • Karma: 34
wdammak écrit:
Il n'est pas instantané et très handicapant pour l'évolution des projets...
des fois il faut 3 PR/3jours/3semaines(si on a rien a foutre dans nos vies) pour ajouter/supprimer un espace avant/apres un "."
c'est vraiment de la merde!

C'est pour une qualité du code justement pour éviter des styles trop différent. Il suffit d'utiliser un logiciel compatible PSR2, c'est plus facile ensuite mais la dernière fois, j'ai quand même eu 5PR pour une erreur travis
Dernière édition: il y a 4 semaines 5 heures par aspangaro-OpenDSI.
L'administrateur a désactivé l'accès en écriture pour le public.

Abandonner Travis! il y a 4 semaines 3 heures #114069

  • wdammak
  • Portrait de wdammak
  • Hors ligne
  • Bannis
  • Messages : 207
  • Remerciements reçus 21
  • Karma: 0
Arre écrit:
euhhh ok,
peux tu préciser ta pensée stp ? autant pour toi et dans ton attitude de "colère" : ça doit avoir du sens.

Autant pour les lecteurs de ton post : on, a aucune idée de quoi tu parles
L'outil qui vérifie les PR sur github
L'administrateur a désactivé l'accès en écriture pour le public.
Cet utilisateur a été remercié pour son message par: Arre

Abandonner Travis! il y a 4 semaines 3 heures #114070

  • wdammak
  • Portrait de wdammak
  • Hors ligne
  • Bannis
  • Messages : 207
  • Remerciements reçus 21
  • Karma: 0
aspangaro-OpenDSI écrit:
wdammak écrit:
Il n'est pas instantané et très handicapant pour l'évolution des projets...
des fois il faut 3 PR/3jours/3semaines(si on a rien a foutre dans nos vies) pour ajouter/supprimer un espace avant/apres un "."
c'est vraiment de la merde!

C'est pour une qualité du code justement pour éviter des styles trop différent. Il suffit d'utiliser un logiciel compatible PSR2, c'est plus facile ensuite mais la dernière fois, j'ai quand même eu 5PR pour une erreur travis
plugin for Notepad++?
Je n'utilise que ce logiciel ..
Dernière édition: il y a 4 semaines 3 heures par wdammak.
L'administrateur a désactivé l'accès en écriture pour le public.

Abandonner Travis! il y a 3 semaines 6 jours #114077

  • aspangaro-OpenDSI
  • Portrait de aspangaro-OpenDSI
  • Hors ligne
  • Modérateur
  • Messages : 1355
  • Remerciements reçus 352
  • Karma: 34
Bonjour,

Dans ce cas désolé, il n'y a pas de solution avec Notepad++ pour le PSR2 à ma connaissance. Depuis 8 mois, j'utilise PHPStorm qui te permet d'utiliser le langage et la norme désiré.

Il faudrait voir du côté d'atom peut être, c'est l'outil proposé par github, il est gratuit mais je ne peux pas t'en dire plus...

Bon courage.
L'administrateur a désactivé l'accès en écriture pour le public.

Abandonner Travis! il y a 3 semaines 6 jours #114098

  • philazerty
  • Portrait de philazerty
  • Hors ligne
  • Gold Boarder
  • Intégrateur Mon-Dolibarr.fr Hébergement Infogéré
  • Messages : 5829
  • Remerciements reçus 882
  • Karma: 113
Salut
Basé sur Atom, il y a "Visual Studio Code" de Microsoft ! Gratuit
@+

Edit : existe pour linux aussi je crois
Dernière édition: il y a 3 semaines 6 jours par philazerty.
L'administrateur a désactivé l'accès en écriture pour le public.

Abandonner Travis! il y a 3 semaines 6 jours #114106

  • defrance
  • Portrait de defrance
  • Hors ligne
  • Gold Boarder
  • Dev-Leader des patas-monkey
  • Messages : 3376
  • Remerciements reçus 668
  • Karma: 132
Je suis passée à phpstorm après plusieurs années d'ultra-edit et même si il s'agit d'un éditeur payant c'est une vrai bonheur de travailler dessus (en particulier avec Symfony, je me demande si on peu réellement coder avec ce framework en dehors de cet éditeur).
Pour en revenir à Travis, j'ai moi aussi beaucoup pesté contre lui à une certaine époque... à présent je fait plus attention à ma manière de coder, la contrainte amène bien souvent la créativité...
L'administrateur a désactivé l'accès en écriture pour le public.

Abandonner Travis! il y a 3 semaines 6 jours #114107

  • dolibarr95
  • Portrait de dolibarr95
  • Hors ligne
  • Admin
  • Messages : 1270
  • Remerciements reçus 188
  • Karma: 25
Bonjour :)
Cela veut dire que vous déposez vos code sur Github en privé (je pense) et faites vérifier par Travis?
L'administrateur a désactivé l'accès en écriture pour le public.

Abandonner Travis! il y a 3 semaines 6 jours #114109

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

non, c'est pas exactement comme ça que ça fonctionne : lis leur doc (ça part d'un bon sentiment, même si je comprend maintenant le côté "frustrant" que ça peut avoir pour les dev)

travis-ci.org/
L'administrateur a désactivé l'accès en écriture pour le public.

Abandonner Travis! il y a 3 semaines 6 jours #114110

  • dolibarr95
  • Portrait de dolibarr95
  • Hors ligne
  • Admin
  • Messages : 1270
  • Remerciements reçus 188
  • Karma: 25
Merci
je m'interresse bcp a travis/github depuis un p'tit bout de temps (pour mon compte perso )et l'idée me séduis mais ce que je comprends pas c'est le prix :S : travis-ci.com/plans 69$ pour des repos privés.
Après pour Dolibarr c'est open source donc pas de soucis c'est gratuit :P et efficace.
L'administrateur a désactivé l'accès en écriture pour le public.

Abandonner Travis! il y a 3 semaines 6 jours #114117

  • defrance
  • Portrait de defrance
  • Hors ligne
  • Gold Boarder
  • Dev-Leader des patas-monkey
  • Messages : 3376
  • Remerciements reçus 668
  • Karma: 132
A titre d'info, j'ai mis en place sur mon environnement de dev un serveur jenkins, totalement gratuit qui permet de vérifier non seulement les PSR mais aussi la qualité du code et d'autres joyeuses choses.
L'administrateur a désactivé l'accès en écriture pour le public.
Cet utilisateur a été remercié pour son message par: Arre, dolibarr95

Abandonner Travis! il y a 3 semaines 6 jours #114118

  • Arre
  • Portrait de Arre
  • Hors ligne
  • Modérateur
  • Envie de participer ? wiki -> adhérer à l'asso :)
  • Messages : 1881
  • Remerciements reçus 428
  • Karma: 44
@defrance
il est amusant (de mon point de vue, et malgré des connaissances assez avancées en info/prog/admin) de lire une phrase sans y comprendre un traître mot, mais de malgré tout la trouver utile :)
L'administrateur a désactivé l'accès en écriture pour le public.
Cet utilisateur a été remercié pour son message par: defrance

Abandonner Travis! il y a 3 semaines 4 jours #114221

  • jtraulle
  • Portrait de jtraulle
  • Hors ligne
  • Admin
  • Messages : 226
  • Remerciements reçus 83
  • Karma: 15
Même si c'est vrai que je trouve saoulant de corriger une PR soumise à chaque fois à cause de problèmes de syntaxe qui ne viennent en plus pas nécessairement des modifications de la PR en question (soit d'anciennes PR qui ont été fusionnées dans les branches alors qu'elles étaient en erreur, soit des devs de la core team qui poussent en direct dans les branches et "squizzent" donc l'intégration continue), ça reste une bonne chose et permet d'éviter que tout le monde (enfin presque) fasse n'importe quoi :)

Pour les personnes qui se demanderaient à quoi sert Travis CI :

Il s'agit d'un logiciel en ligne qui fonctionne avec Github.

Pour rappel, lorsqu'un utilisateur de Dolibarr souhaite intégrer des modifications qu'il a apporté au code source de Dolibarr, il lui est possible de faire une Pull Request ou PR sur Github (une Pull Request, c'est dans le "jargon" Github, une demande aux mainteneur d'un projet sur Github d'intégrer les modifications que l'on a effectué au code source dans le projet principal).

Afin de garder un certain niveau de qualité au niveau du code source d'un logiciel, il est possible de mettre en place ce que l'on appelle de l'intégration continue. L'intégration continue permet d'exécuter de façon automatique un certain nombre d'actions à chaque fois qu'une Pull Request est soumise ; par exemple :
  • vérifier que le code ne produit pas d'erreurs de syntaxe à l'exécution ; pour cela, on peut utiliser par exemple phplint
  • vérifier que les fichiers sont conformes à une convention de codage ou à un standard de style de code (PSR2 [en]) ; pour cela on peut utiliser par exemple PHP_CodeSniffer
  • exécuter des tests unitaires et/ou des tests d'intégration et vérifier qu'ils ne retournent aucune erreur : on peu utiliser par exemple phpunit

Évidement, ces différentes actions prennent du temps à être exécutés et le sont à chaque soumission d'une Pull Request, ce qui peut expliquer l'énervement de certains ... De plus, les différentes actions sont exécutées pour différentes versions de PHP et différents SGBDR (Système de Gestion de Bases de Données Relationnels) à chaque fois. Actuellement chaque Pull Request est testée avec l'ensemble de la suite d'actions (lint, phpcs, tests) pour PHP 5.5, 7.3 et nightly et pour MySQL (MariaDB) et PostgreSQL.

Travis-CI est donc l'outil qui exécute ces actions de tests. Si vous êtes curieux, vous pouvez accéder aux tests de toutes les PR ici : travis-ci.org/dolibarr/dolibarr/pull_requests

En définitive, l'intégration continue est un passage obligé pour garantir un certain niveau de qualité sur un projet logiciel. Il est possible de la mettre en œuvre de diverses manières et en utilisant diverses solutions techniques/logicielles (j'utilise par exemple personnellement Gitlab CI qui est intégré dans Gitlab : une alternative libre/opensource à Github).

Il ne faut pas perdre de vue que l'exécution des tests consomme "du temps machine" et que c'est déjà une chance d'avoir la possibilité d'utiliser un outil comme Travis CI gratuitement pour les projets Open source tels que Dolibarr.

L'ensemble des actions exécutées sont consultables ici : github.com/Dolibarr/dolibarr/blob/develop/.travis.yml et il reste toujours possible de les exécuter localement AVANT de soumettre sa PR pour éviter d'avoir à la corriger ensuite suite à un échec.

Voir aussi : How to run travis-ci locally [en]

Désolé pour la tartine ... j'espère que ça aura été instructif pour quelques uns ;)
Dernière édition: il y a 3 semaines 4 jours par jtraulle.
L'administrateur a désactivé l'accès en écriture pour le public.
Cet utilisateur a été remercié pour son message par: aspangaro-OpenDSI, yves57, wdammak, dolibarr95, ksar, pcbleu

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

  • yves57
  • Portrait de yves57
  • Hors ligne
  • Gold Boarder
  • Messages : 666
  • Remerciements reçus 138
  • Karma: 11
Voici des explications fort intéressantes.
Au point qu'elles méritent une place sur le wiki. :p
Par exemple pour compléter cette page. wiki.dolibarr.org/index.php/Outils_de_d%C3%A9veloppement
J'apprends la solution du travis local. A priori, ça semble du lourd, donc pour quelqu'un qui est en permanence sur le truc, pas pour de l'occasionnel.
Je pense qu'il faut des solutions intermédiaires, comme les éditeurs cités précédemment.
L'administrateur a désactivé l'accès en écriture pour le public.
  • Page :
  • 1
  • 2