Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Mise à jour des dépendances Elixir #3676

Merged
merged 4 commits into from
Dec 21, 2023
Merged

Conversation

AntoineAugusti
Copy link
Member

@AntoineAugusti AntoineAugusti commented Dec 20, 2023

J'ai lancé mix deps.update --all.

Oban

J'ai suivi le guide Oban pour 2.17. insert_trigger: false me parait adapté à notre usage.

Disable notifications with the insert_trigger option if sub-second job execution isn't important or you'd like to reduce PubSub chatter

J'ai fait les changements de format pour unique (passant de secondes à un tuple plus explicite).

Le changement de Notifier ne me parait pas nécessaire à effectuer comme on utilise peu ce mécanisme (on l'utilise dans quelques liveviews dans le BO pour savoir qu'un job est terminé).

gettext

Il manquait quelques elixir-autogen dans les fichiers *.po ce qui levait une erreur dans mix gettext.extract --check-up-to-date. On aurait dû traiter ça un peu plus tôt.

Autres dépendances

Après une lecture des changelogs/diffs je ne vois pas d'autres choses à modifier.

Changelogs

@AntoineAugusti AntoineAugusti marked this pull request as ready for review December 20, 2023 14:12
@AntoineAugusti AntoineAugusti requested a review from a team as a code owner December 20, 2023 14:12
Copy link
Contributor

@thbar thbar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Well done!

Un premier commentaire "rapide" pour voir "ce qui reste" après, j'utilise souvent ceci pour voir ce qui est "bloqué" par la façon dont on spécifie nos versions:

❯ mix hex.outdated | grep -v "Up-to-date"
Dependency              Current  Latest  Status               
mox                     1.0.2    1.1.0   Update not possible  
phoenix                 1.6.16   1.7.10  Update not possible  
phoenix_html            3.3.3    4.0.0   Update not possible  
phoenix_live_dashboard  0.7.2    0.8.3   Update possible      
phoenix_live_view       0.18.18  0.20.2  Update not possible  
postgrex                0.16.5   0.17.4  Update not possible  
sentry                  8.1.0    10.1.0  Update not possible  
ymlr                    4.2.0    5.0.0   Update not possible  

Run `mix hex.outdated APP` to see requirements for a specific dependency.

To view the diffs in each available update, visit:
https://hex.pm/l/WMw37

La commande utilisée:

J'ai lancé mix deps.update --all.

Va respecter les contraintes dans mix.exs et donc rater quelques trucs.

Suggestion:

  • Voir si certains ne peuvent pas être mis à jour (voire, "relaxées" en terme de contrainte, au moins tant que ça n'est pas trop majeur - ex Mox peut-être)
  • Sinon créer des tickets pour le reste (ex: si ymlr introduit trop de casse, mais c'est peut-être totalement digérante)

Comme ça ça permettra de faire une seule mise en prod "plus grosse" et d'avoir plus de choses à jour.

@AntoineAugusti
Copy link
Member Author

@thbar J'ai bump Mox en plus. Le CHANGELOG ne posait pas de problème.

Je traiterai le reste dans une autre PR (phoenix_* a déjà une issue ouverte) et en regardant rapidement Sentry (au moins) requiert plus de temps.

@AntoineAugusti AntoineAugusti requested a review from thbar December 20, 2023 15:43
Copy link
Contributor

@thbar thbar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@AntoineAugusti good!

Je suis partagé sur le fait de valider et d'envoyer là, en même temps il reste jeudi et vendredi, ça laisse le temps de se retourner, et les problèmes sont rares.

Je me demande si on préfère tester rapidement sur prochainement aussi ?

Let's go otherwise!

@AntoineAugusti AntoineAugusti added this pull request to the merge queue Dec 21, 2023
Merged via the queue into master with commit c16d891 Dec 21, 2023
@AntoineAugusti AntoineAugusti deleted the 202312_update_deps branch December 21, 2023 09:27
@AntoineAugusti
Copy link
Member Author

AntoineAugusti commented Dec 28, 2023

La mise à jour de postgrex et de ymlr est impossible pour le moment. J'ai ouvert des issues pour s'en souvenir.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants