opt-nc / opt-logging

La librairie de référence pour générer des logs bien formatées à l'OPT.
0 stars 1 forks source link

Upgrade to SB v2.6.4 #78

Closed adriens closed 2 years ago

adriens commented 2 years ago
Dougniel commented 2 years ago
  • [ ] Faire une comm' sur comment s'abonner aux news de opt-logging (@adriens ), et communqiuer sur arrivée prochaine de la 2.7.0

@adriens, On n'avait pas dit que opt-logging était dépréciée ?

adriens commented 2 years ago

Les devs onPrem l'utllisent toujurs puisque sur le socle

adriens commented 2 years ago

l'idée est donc de leur fournir une vesion maintenue et d'aculturer les gens à suivre/monitorer les montées de versions de leurs dépendances... même sans dependabot

Dougniel commented 2 years ago

Les devs onPrem l'utllisent toujurs puisque sur le socle

D'accord, mais de là à faire de la comm dessus, je trouve que cela va dans la promotion de ce composant et ça nous donne une comm incohérente

l'idée est donc de leur fournir une vesion maintenue et d'aculturer les gens à suivre/monitorer les montées de versions de leurs dépendances... même sans dependabot

Je comprends, mais je pense qu'il faut trouver un autre sujet

adriens commented 2 years ago

C'est pas incohérent, cette lib est utilisée en masse à l'OPT. Et on maintient ce que l'on utilise. Il n'y pas que nous...et c'est pas demain que tous les devs seront sur GH comme nous en avons la chance > Ne pas oublier ça.

D'autres sujets viendront compléter. Mais pour l'instant, ce sujet sera poussé. Un investissement a été effectué, pour montrer la continuité. C'est didactique.

adriens commented 2 years ago

J'attends les upgrades et release donc stp.

Dougniel commented 2 years ago

@adriens, il n'y a pas de spring-boot car il s'agit d'une lib, par contre il y a une dépendance spring-context qui a été upgradé à la dernière version ce qui signifie que ça ne devrait pas poser pb avec les projets qui vont utiliser cette dépendance

NOTE: la dependance springboot-starter-test a été utilisée pour les TU, mais je n'en vois pas l'intérêt : junit aurait suffit

Dougniel commented 2 years ago

Bon galère 😅 semantic-release :

image

Alors que :

image

et :

image
Dougniel commented 2 years ago

La raison au fait que la release GitHub n'est pas été générée est ici : https://github.com/opt-nc/opt-logging/commit/a227cdff578fcc055a40a9be5ac72feba6be615b

J'ai fait une issue la dessus à @mbarre car je ne sais pas si c'était volontaire ou non ou quels ont été les pb que @mbarre a rencontré : https://github.com/opt-nc/opt-logging/issues/81

En attendant, j'ai généré la release manuellement

adriens commented 2 years ago

Bon galère 😅 semantic-release

Oui, on dirrait bien cf https://github.com/opt-nc/conventional-commit/pull/34

adriens commented 2 years ago

En attendant, j'ai généré la release manuellement

👍

adriens commented 2 years ago

Comm' faite 👍