These are chat archives for UJM-dev/ExoBundle

11th
Sep 2015
Guillou Patrick
@pitrackster
Sep 11 2015 07:12
Salut ! on essaye d'installer l'ujm-exo bundle à partir du package 6.x-dev qui a les dépendances angular dans son composer... mais composer gueule en disant qu'il ne trouve pas le package claroline/core-bundle ^6.2.5... je ne trouve pas de version tagguée 6.x-dev dans github donc pas moyen de vérifier le composer associé... qq'un sait ? Merci !
Gruber Laurent
@LaurentGruber
Sep 11 2015 07:15
il faudrait mettre comme require pour le corebundle plutôt ~6.0
on a encore tagger le core hier
on est à la 6.3.1
Stéphane Klein
@stefk
Sep 11 2015 07:15
le tag 6.2.5 existe
Guillou Patrick
@pitrackster
Sep 11 2015 07:16
yep mais la syntaxe ne parrait pas marcher...
Gruber Laurent
@LaurentGruber
Sep 11 2015 07:16
oui mais idéalement il faudrait tester avec la dernière version du core à chaque fois
Stéphane Klein
@stefk
Sep 11 2015 07:17
^6.2.5 = ">=6.2.5, <7.0"
Guillou Patrick
@pitrackster
Sep 11 2015 07:17
c'est pour ça que je voulais voir le composer.json de la version 6.x-dev pour voir comment c'est écrit dedans... cette syntaxe devrait permettre de mettre le core bundle le plus récent
oups stefk a répondu avant moi
Guillou Patrick
@pitrackster
Sep 11 2015 07:22
il est sur quel repo ce tag de l'exo-bundle ?
6.x-dev
Stéphane Klein
@stefk
Sep 11 2015 07:25
pas sûr de bien comprendre... le seul tag lié à la v6 sur l'exo bundle, c'est 6.0.0
6.x-dev, j'imagine que ça doit être un alias de branche pour composer
mais je ne vois d'où il soirt
sort
il n'y a pas de branche 6.0 dans l'exo bundle
Guillou Patrick
@pitrackster
Sep 11 2015 07:26
bah oui c'est pour le moins étrange !!
Stéphane Klein
@stefk
Sep 11 2015 07:27
où est-ce que tu vois "6.x-dev" ?
Guillou Patrick
@pitrackster
Sep 11 2015 07:27
dans packagist pour l'exo-bundle
sinon on va se démerder autrement et mettre les dépendances qui vont bien à la main
sur une autre version
Gruber Laurent
@LaurentGruber
Sep 11 2015 07:54
on pourrait vers un tag v6 pour tester plus facilement
enfin plus tôt v6 dev
non ?
Stéphane Klein
@stefk
Sep 11 2015 07:56
tu veux dire sur l'exo bundle ?
Gruber Laurent
@LaurentGruber
Sep 11 2015 08:05
oui
Stéphane Klein
@stefk
Sep 11 2015 08:11
Bon, je crois que je commence à comprendre... La branche "v6" est interprétée comme une vraie branche 6.x. "6.x-dev" est une contrainte de version qui veut dire "la derniere version de la branche 6". Ça ne doit pas correspondre à un tag ou à une branche ayant ce nom. Autrement dit, à partir du moment où on a une branche "6" ou "6.0" ou "6.0.0" ou "v6" ou "V6" etc., on peut utiliser 6.x-dev comme contrainte de version et ça pointera sur le dernier commit de la branche.
Un super outil pour vérifier: http://semver.mwl.be/
Maintenant le problème, c'est que le composer.json de la branche 6 de l'exo bundle requiert l'icap bundle en version 5...
Il y a un conflit sur la version du core bundle
@purplefish32, lol, 30 rock, ça faisait longtemps
Gruber Laurent
@LaurentGruber
Sep 11 2015 08:15
ok donc il faut corriger le composer.json de la branche 6 ?
Stéphane Klein
@stefk
Sep 11 2015 08:16
oui
Gruber Laurent
@LaurentGruber
Sep 11 2015 08:16
c'est pratique comme système de gestion des branches
tu t'y colle ou tu veux que je le fasse ?
Stéphane Klein
@stefk
Sep 11 2015 08:16
pareil
je suis sur un merge là, donc si t'as le tmps...
Gruber Laurent
@LaurentGruber
Sep 11 2015 08:17
oki, je vais le faire direct sur github
Stéphane Klein
@stefk
Sep 11 2015 08:17
ok
Gruber Laurent
@LaurentGruber
Sep 11 2015 08:19
c'est fait, si quelqu'un de UJM peut faire le merge
Stéphane Klein
@stefk
Sep 11 2015 08:20
la v6 du bundle peut pas fonctionner avec un core en-dessous de 6.2.5
il faut laisser la contrainte ^6.2.5 ou en mettre une équivalente
Stéphane Klein
@stefk
Sep 11 2015 08:57
@pitrackster, tu utilises un outil de mise en forme automatique du code ?
Guillou Patrick
@pitrackster
Sep 11 2015 08:59
oui...
Stéphane Klein
@stefk
Sep 11 2015 08:59
tu le configures avec quel standard de codage ?
Guillou Patrick
@pitrackster
Sep 11 2015 09:00
jamais configuré... c'est Netbeans... je vais zieuter... y'a des trucs tout pourrits ?
Stéphane Klein
@stefk
Sep 11 2015 09:00
ouaip...
enfin c'est surtout que c'est pas conforme au standard de symfony par endroit
Guillou Patrick
@pitrackster
Sep 11 2015 09:01
arf je vais zieuter
j'ai des options mais du coup tu as un lien vers les standards symfony pour que je puisse modifier tout ça ?
une grosse différence c'est le { qui passe à la ligne pour les defs de classes et de méthodes
Guillou Patrick
@pitrackster
Sep 11 2015 09:05
ouaip je vois... je modifie ça dans mes options de formattage automatique...
Stéphane Klein
@stefk
Sep 11 2015 09:06
t'as choisi le formattage à la sauvegarde du fichier ?
Guillou Patrick
@pitrackster
Sep 11 2015 09:06
non par raccourcis de touche
je l'utilise essentiellement pour réindenter correctement mon code... j'avais jamais fait gaffe à ces standards
Stéphane Klein
@stefk
Sep 11 2015 09:07
mais ça l'applique sur tout le code du fichier que tu édites, non ?
Guillou Patrick
@pitrackster
Sep 11 2015 09:08
oui... j'ai tendance à appliquer le formattage auto un peu systématiquement...
Stéphane Klein
@stefk
Sep 11 2015 09:09
le pb, c'est que potentiellement, pour un simple changement, tout le fichier est modifié et ça devient très compliqué dans les diffs...
perso je préfére les commits séparés dans ces cas-là
Guillou Patrick
@pitrackster
Sep 11 2015 09:11
ouaip je comprends... pour les futurs commit on fera des pull request à partir de branches locales... mais ça risque de poser les mêmes problèmes si on formatte notre code différemment... ce problème devrait être réglé par les modifs que j'ai faites dans mes options de formattage ("{" à la ligne pour déclarations de classes et de méthodes)...
du coup y'a une meilleur solution ? tu fais comment de ton côté ? tu fais tout à la main ?
Stéphane Klein
@stefk
Sep 11 2015 09:13
moi... ben je suis un maniaque qui fait tout manuellement...
Guillou Patrick
@pitrackster
Sep 11 2015 09:13
lol... en même temps ça a l'air d'être la seule solution qui marche ;)
Stéphane Klein
@stefk
Sep 11 2015 09:14
par contre il y a un outil qui pourrait être utile : http://cs.sensiolabs.org/
on pourrait passer un coup sur tout le code dans une PR, puis l'utiliser en automatique pour ceux qui veulent
Guillou Patrick
@pitrackster
Sep 11 2015 09:15
ouaip ça a l'air d'être bien pour corriger après coup...
oui
ça serait nickel !!
je l'installe et voit ce que ça donne
Stéphane Klein
@stefk
Sep 11 2015 09:16
@UJM-dev, @charlotte42, vous en pensez quoi ?
Guillou Patrick
@pitrackster
Sep 11 2015 09:50
j'ai passé la chose sur le bundle... ça à l'air pas mal... et puissant... je fais une pull request pour que vous permettre de voir le résultat ?
Stéphane Klein
@stefk
Sep 11 2015 09:51
ok !
Guillou Patrick
@pitrackster
Sep 11 2015 09:55
c'est fait
Donovan Tengblad
@purplefish32
Sep 11 2015 10:08
This message was deleted
Guillou Patrick
@pitrackster
Sep 11 2015 12:30
@stefk du coup le composer ressemblerait à ça c'est correcte ? (je demande parce que je sais plus ce qui a été fait pour la version du core-bundle qu'on doit mettre dans le composer) "require": { "claroline/core-bundle": "^6.2.5", "icap/badge-bundle": "~6.0", "innova/angular-js-bundle": "~5.1", "innova/angular-ui-bootstrap-bundle" : "~5.0", "innova/angular-ui-tinymce-bundle" : "~5.0", "innova/angular-ui-translation-bundle" : "~5.0", "innova/angular-ui-resource-picker-bundle" : "~6.0", "innova/angular-ui-sortable-bundle" : "~5.0", "innova/angular-ui-pageslide-bundle" : "~5.0" }, "require-dev": { "phpunit/phpunit": "~4.6", "fabpot/php-cs-fixer":"^1.10" }
Stéphane Klein
@stefk
Sep 11 2015 12:32
ouaip
par curiosité, c'est quoi la version du fixer que tu as chez toi ?
Guillou Patrick
@pitrackster
Sep 11 2015 12:33
la 1.10
Stéphane Klein
@stefk
Sep 11 2015 12:34
et t'as utilisé quelle méthode d'install ?
Guillou Patrick
@pitrackster
Sep 11 2015 12:46
Stéphane Klein
@stefk
Sep 11 2015 12:48
ok. composer ramenera la 1.11, mais ça ne devrait pas changer grand chose...
Guillou Patrick
@pitrackster
Sep 11 2015 13:13
PR mise à jour...