etalab / transport-site

Rendre disponible, valoriser et améliorer les données transports
https://transport.data.gouv.fr
198 stars 30 forks source link

Mise à jour librairie de gestion des timezones (`tzdata`) #4177

Closed thbar closed 2 months ago

thbar commented 2 months ago

Dans cette PR:

Erreur traitée

J'ai vu en local plusieurs fois l'erreur suivante, que j'avais laissé de côté pour le moment :

[error] GenServer :tzdata_release_updater terminating
** (ArgumentError) errors were found at the given arguments:

  * 2nd argument: not a tuple

    :erlang.element(1, :error)
    (tzdata 1.1.1) lib/tzdata/util.ex:223: Tzdata.Util.to_int/1

Plus tard dans mon flux twitter est apparu:

Je fais donc l'upgrade correspondante, mais vu la fraîcheur de la release (il y a 8 heures), je propose de patienter un peu avant de merger, voir si il n'y a pas de "contre-release" :smile:

Changelogs

(initially generated with elixir generate_deps_changelogs.exs)

AntoineAugusti commented 2 months ago

J'ai tenté de comprendre le %z de tzdata mais difficilement comme abbréviation https://github.com/lau/tzdata/pull/147

thbar commented 2 months ago

Je ne vois pas de nouvelle release en mode panique (https://github.com/lau/tzdata/tags), je mergerai demain matin.

Un point intéressant à suivre:

thbar commented 2 months ago

@etalab/transport-tech je déploie cette PR. Si vous voyez des trucs chelous (cf fragilité remontée ici https://github.com/lau/tzdata/issues/149), faites signe !

ptitfred commented 2 months ago

@etalab/transport-tech je déploie cette PR. Si vous voyez des trucs chelous (cf fragilité remontée ici lau/tzdata#149), faites signe !

Well j'observe une volée de warnings causés par Code.eval_string/1 dans le test rajouté dans test/build_test.exs. Extrait :

warning: found quoted keyword "ymlr" but the quotes are not required.

(Le mix.lock quote les clefs en effet.)