Correctifs pour OTP 26+ #4286
Correctifs pour OTP 26+ #4286
Conversation
thbar
added
the
dette technique
There was a problem hiding this comment.
Le test apps/datagouvfr/test/datagouvfr/client/discussions_test.exs à changer, mais à part ça c’est bon.
Ça ne résout pas tous les problèmes d’ordre de map sur ma machine (AMD Ryzen 7 + Fedora), mais comme vu ensemble ça pourra faire l’objet d’une PR séparée.
Yes, j'ai traité dans fd089c5. Merci ! Je te renvoie la PR pour review du coup.
Tout à fait, il va y en avoir d'autres. Je "backporte" petit à petit de #3993. |
| count_entities | ||
| |> Map.filter(fn {_, v} -> v > 0 end) | ||
| |> Enum.map(fn {k, _} -> Atom.to_string(k) end) | ||
| |> Enum.sort() |
There was a problem hiding this comment.
Ce qui change c'est le sort à la fin, le reste est du reformattage.
Cette PR corrige 5 tests et 1 méthode du code réel pour que le tout fonctionne avec OTP 27. Il y en aura d'autres.
Tiré de:
Explication de pourquoi il faut faire ça
(emphasis mine)
Voir: