Ticket #5056 Premiers pas Composer #5057
Merged
b_b
merged 21 commits from issue_5056_composer_road
into master
6 months ago
Loading…
Reference in new issue
There is no content yet.
Delete Branch 'issue_5056_composer_road'
Deleting a branch is permanent. It CANNOT be undone. Continue?
On charge ici l’autoloader de Composer (vendor/autoload.php), ainsi que des petites dépendances (polyfill php 8.0 8.1 et mbstring).
On fait évoluer le code
@JamesRezo je vois un double
/plugins/ export-ignore
dans53c982a2d4
;)Je me questionne aussi sur le
/config/
vu qu’on a un fichierecran_securite.php
dedans pour le moment ?bien vu ! c'est corrigé, merci :-)
C'est bon comme ça : .gitattributes exclut /config et composer.json archive.exclude le réintègre
ah je me rappelle, j'avais rien relu ni approuvé car la page de ce diff avait fait exploser mon navigateur (et bon là j'ai pas tout pu lire vu la quantité et même ya pas tout "Certains fichiers n'ont pas été affichés car il y a eu trop de fichiers modifiés dans ce diff")
Même si gitea nous force à afficher le diff avant de pouvoir valider un PR, j'y suis arrivé :)
J'ai bien envie de cliquer sur le bouton de fusion... :)
J’ai rebasé, squashé un peu plus, up les libs composer, et j’ai tenté de mettre des entête à la https://www.conventionalcommits.org/fr/v1.0.0/
Ceci dit je n’ai pas vu de problème apparent là pour le moment.
Ça fait plusieurs fois avec Gitea là que je dois pousser mes commits (push --force) plusieurs fois : il fait comme si tout fonctionnait la première fois (on voit les notifs ici, et sur le bot shiraz), mais… pouf c’est toujours les anciens commits qui sont dans la branche pourtant.
Je dois push une seconde fois pour que ça passe.
Quelle est donc cette fourberie de machinerie ?
On aura quelques menus soucis du coup avec les plugins qui utilisaient les "classes" du compilo, qui sont maintenant dans un namepsace :
ex
saisies/balise/saisie.php Line 207 : Class "Champ" not found
https://git.spip.net/spip-contrib-extensions/saisies/src/branch/master/balise/saisie.php#L207
Au lieu de
new Champ
il faudranew \Spip\Core\Balise
(ou Champ (alias), mais Balise sera plus clair pour tout le monde je crois).Il faudrait d’ailleurs trouver un meilleur namespace pour ces éléments du compilateur…
Spip\Template\Node
?Spip\Compiler\Node
?Spip\Parser\Node
?Spip\Engine\Parser\Node
?Quelques usages sur la zone
Dans les dépots de spip-contrib-extensions, on trouve
Hum, du coup il faudra tester la version de SPIP pour appeler le bon code ? Pas moyen d'avoir un autre mécanisme de compatiblitéé ?
tu dois pouvoir faire un
class_exists("\Spip\Core\Balise")
pour savoir si tu dois instancier la nouvelle classe ou l'ancienneIl faut probablement ajouter
disallow: /vendor
(et d'autres ?) aurobots.txt.html
n'est il pas ?Bon, on peut dire que c'est bon maintenant, amha il est temps de merger :)
137dabdb55
into master 6 months agoReviewers
137dabdb55
.