Ticket #5056 Premiers pas Composer #5057

Open
marcimat wants to merge 14 commits from issue_5056_composer_road into master
Owner

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

  • sur certains strpos() en passant.
  • sur la présence de mb_*
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 - sur certains strpos() en passant. - sur la présence de mb_*
marcimat added 4 commits 4 months ago
528b8fddd4 On charge l’autoloader de composer.json aux 3 points d’entrées de SPIP
f702a0e3e4 On charge et utilise le polyfill-mbstring. Les fonctions mb_ sont donc toujours disponibles.
JamesRezo added 6 commits 4 months ago
JamesRezo added 1 commit 4 months ago
JamesRezo added 1 commit 4 months ago
Owner

@JamesRezo je vois un double /plugins/ export-ignore dans 53c982a2d4 ;)

@JamesRezo je vois un double `/plugins/ export-ignore` dans 53c982a2d4f3d7b04b2e4c2c6c9285aa67c20f55 ;)
Poster
Owner

Je me questionne aussi sur le /config/ vu qu’on a un fichier ecran_securite.php dedans pour le moment ?

Je me questionne aussi sur le `/config/` vu qu’on a un fichier `ecran_securite.php` dedans pour le moment ?
JamesRezo added 1 commit 4 months ago
Owner

@JamesRezo je vois un double /plugins/ export-ignore dans 53c982a2d4 ;)

bien vu ! c'est corrigé, merci :-)

> @JamesRezo je vois un double `/plugins/ export-ignore` dans 53c982a2d4f3d7b04b2e4c2c6c9285aa67c20f55 ;) bien vu ! c'est corrigé, merci :-)
Owner

Je me questionne aussi sur le /config/ vu qu’on a un fichier ecran_securite.php dedans pour le moment ?

C'est bon comme ça : .gitattributes exclut /config et composer.json archive.exclude le réintègre

> Je me questionne aussi sur le `/config/` vu qu’on a un fichier `ecran_securite.php` dedans pour le moment ? > C'est bon comme ça : .gitattributes exclut /config et composer.json archive.exclude le réintègre
JamesRezo added 1 commit 4 months ago
marcimat added 1 commit 4 months ago
marcimat added 15 commits 4 months ago
2532e9770b On charge l’autoloader de composer.json aux 3 points d’entrées de SPIP
fbdd3c3e7e On charge et utilise le polyfill-mbstring. Les fonctions mb_ sont donc toujours disponibles.
marcimat added 15 commits 4 months ago
63c3e2d188 On charge l’autoloader de composer.json aux 3 points d’entrées de SPIP
30fd65dfec On charge et utilise le polyfill-mbstring. Les fonctions mb_ sont donc toujours disponibles.
marcimat added 1 commit 4 months ago
JamesRezo added 1 commit 4 months ago
JamesRezo added 3 commits 4 months ago
JamesRezo added 1 commit 4 months ago
JamesRezo added 1 commit 3 months ago
marcimat added 22 commits 3 months ago
a7d4d77cd8 On charge l’autoloader de composer.json aux 3 points d’entrées de SPIP
fc84ff2723 On charge et utilise le polyfill-mbstring. Les fonctions mb_ sont donc toujours disponibles.
rastapopoulos approved these changes 2 months ago
rastapopoulos left a comment
Owner

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")

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")
b_b approved these changes 2 months ago
b_b left a comment
Owner

Même si gitea nous force à afficher le diff avant de pouvoir valider un PR, j'y suis arrivé :)

Même si gitea nous force à afficher le diff avant de pouvoir valider un PR, j'y suis arrivé :)
Owner

J'ai bien envie de cliquer sur le bouton de fusion... :)

J'ai bien envie de cliquer sur le bouton de fusion... :)
marcimat added 22 commits 1 month ago
3b2b76e785 On charge l’autoloader de composer.json aux 3 points d’entrées de SPIP
fa6d20a08b On charge et utilise le polyfill-mbstring. Les fonctions mb_ sont donc toujours disponibles.
marcimat added 14 commits 4 weeks ago
4803f9332a feat(autoloader): Charger l’autoloader de Composer aux 3 points d’entrées de SPIP
ee366ca9ed refactor: Charger et utiliser le polyfill-mbstring.
Poster
Owner

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/

  • Il manque encore les entrées de changelog
  • Je me demande s’il faudrait finalement pas attendre (ou nécessiter) PHP 8.0 mini : j’ai peur que certaines libs ne soient pas entièrement compatibles sur la plage PHP 7.4 à PHP 8.1 (notamment sur la correction des deprecated)

Ceci dit je n’ai pas vu de problème apparent là pour le moment.

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/ - Il manque encore les entrées de changelog - Je me demande s’il faudrait finalement pas attendre (ou nécessiter) PHP 8.0 mini : j’ai peur que certaines libs ne soient pas entièrement compatibles sur la plage PHP 7.4 à PHP 8.1 (notamment sur la correction des deprecated) Ceci dit je n’ai pas vu de problème apparent là pour le moment.
marcimat added 14 commits 4 weeks ago
4803f9332a feat(autoloader): Charger l’autoloader de Composer aux 3 points d’entrées de SPIP
ee366ca9ed refactor: Charger et utiliser le polyfill-mbstring.
Poster
Owner

Ç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 ?

Ç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 ?
goldstein requested review from rastapopoulos 3 weeks ago
goldstein requested review from b_b 3 weeks ago
b_b approved these changes 3 weeks ago
rastapopoulos approved these changes 3 weeks ago

Reviewers

b_b approved these changes 3 weeks ago
rastapopoulos approved these changes 3 weeks ago
This pull request has changes conflicting with the target branch.
ecrire/public/references.php
ecrire/req/sqlite_generique.php
Sign in to join this conversation.
Loading…
There is no content yet.