Intéressant, et je partage le constat (même si ça peut ou a pu m’arriver).
Si je comprends bien, ça crée un tag x.y.z si la chaîne « x.y.z » est trouvée dans le log de commit et que le tag n’existe pas.
Il manque un test sur la présence de « build » en tout début de log, ou bien cette config le teste aussi ? (je n’ai pas une grosse connaissance des configs de CI)
Est ce qu’on aurait des risques de faux positifs ? Les logs qui commencent par build sont toujours très courts, peu de risque à mon avis.
Peut-être on pourrait mettre ça en test sur spip-contrib-extensions ?
Pour un néophyte, je comprends que le tag est généré automatiquement si le message de Commît commence par un build ? Si c’est ça, ça me paraît pas mal, sinon j’ai rien compris
Ça peut sembler magique mais ça ne sera pas miracle car d’une certaine manière c’est passer de « penser à mettre un tag » à « penser un mettre un build »…
Pour moi ça ne sera pas une aide car ma géométrie neuronale me rend imperméable aux conventional commits, mais je pense/j’imagine que tous les adeptes de ce formalisme peuvent apprécier un tel automatisme.
Ce n’est pas qu’une question de conventional commit, en fait.
C’est assez limitant en cas d’erreur ou d’oubli, par exemple. Si on oubli un ; quelque part et qu’il faut re-release, éventuellement effacer le tag, refaire un truc de commit sans vexer le script de CI, c’est chaud …
D’autre part, c’est remplacer un truc obligatoire par un autre, comme tu dis. Et appliquer ça au 1000+ projets git communautaire, ça me semble un pas un peu trop grand. Il faudra peut-être enfin envisager de créer des organisations où des plugins et des mainteneureuses sont ok pour appliquer des règles communes. Ex: plugins d’e-commerce, plugins de gestion de formulaires, collection de plugins sans aucune règles de dev … et déplacer les plugins existants correspondants de spip-contrib-extensions dans ces nouvelles organisations, avec les mainteneureuses correctement identifiées.
Bref, une chose est sûre, spip-releases fait se genre de boulot plutôt bien, c’est à la main des dev/mainteneureuses, c’est responsabilisant, a contrario de ce genre de scripts que je trouve aussi « trop magique ».