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.

Sign up for GitHub

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

Jump to bottom

Mise à jour Elixir 1.10 -> 1.12 et OTP 23 -> 24 #1683

Merged
merged 24 commits into from
Jul 9, 2021
Merged

Mise à jour Elixir 1.10 -> 1.12 et OTP 23 -> 24 #1683

merged 24 commits into from
Jul 9, 2021

Conversation

thbar
Copy link
Contributor

@thbar thbar commented Jun 24, 2021 •
edited
Loading

Le système de build de hex.pm a publié les nouvelles versions et je peux donc tout upgrader d'un coup.

Tâches prévues

  • Créer l'image Docker (OK)
  • Pousser l'image sur Docker Hub (OK)
  • Lancer les tests en local
  • Corriger Dialyzer
    • Postgrex
    • comment checker
  • Analyser les logs et créer les tickets éventuellement associés [WIP] Liste des warnings Elixir #1691 ; une partie est traitée, on va essayer de déployer comme ça
  • Déployer sur prochainement
  • Vérifier le comportement (backoffice, oauth, import des données) sur prochainement
  • Déployer sur la production sans merger pendant quelques jours

@thbar thbar changed the title [WIP] Mise à jour Elixir & OTP Mise à jour Elixir 1.10 -> 1.12 et OTP 23 -> 24 Jun 28, 2021
@thbar thbar marked this pull request as draft June 28, 2021 17:17
@fchabouis
Copy link
Contributor

J'ai regardé l'erreur dialyxir, j'ai essayé de googler ça mais je sèche.

Je ne comprends même pas ce qui l'erreur veut dire en fait : The call _.'id'/() requires that _@2 is of type atom(), not {map(), map()}.

On pourra peut-être regarder ensemble, sinon il faudra qu'on appelle à l'aide sur un forum.

@thbar thbar mentioned this pull request Jul 1, 2021
Closed
19 tasks
This was referenced Jul 6, 2021
Merged
Merged
Merged
Merged
thbar added 2 commits July 6, 2021 15:02
@thbar
Merge branch 'master' into upgrade-elixir-otp
b4a9b4a
@thbar
Update Vex to master for Elixir 1.11 compat
4c7120f 
I also removed vex dependency for apps which did not depend on vex.
@thbar thbar mentioned this pull request Jul 6, 2021
Merged
@thbar thbar requested review from fchabouis and andlio July 8, 2021 17:10
@thbar thbar marked this pull request as ready for review July 8, 2021 17:11
@thbar
Copy link
Contributor Author

thbar commented Jul 9, 2021

@andlio si tu peux faire une review, ça serait top, je mergerai et reconfigurerai CleverCloud juste après. Merci !

@thbar
Copy link
Contributor Author

thbar commented Jul 9, 2021

@fchabouis je te laisse jeter un oeil aussi, vu que c'est un gros truc, et je déploierai après !

fchabouis
fchabouis approved these changes Jul 9, 2021
View reviewed changes
Copy link
Contributor

@fchabouis fchabouis left a comment

Choose a reason for hiding this comment

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

👍

@thbar thbar merged commit d73662f into master Jul 9, 2021
@thbar thbar deleted the upgrade-elixir-otp branch July 9, 2021 12:32
@thbar thbar mentioned this pull request Jul 12, 2021
Closed
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@fchabouis fchabouis fchabouis approved these changes

@andlio andlio andlio approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@thbar @fchabouis @andlio