Skip to content

Upgrade to Python 3.7 / Django 2.1

Nim requested to merge Nim/coin:master into master

Hello !

Ce WE à l’AG, j’ai tenté une petite mise à jour. J’avais pas vu !24 (merged)

Bref, j’ai commencé par mettre à jour petit à petit toutes les dépendances que j’ai pu en m’assurant que les tests unitaires passaient encore, jusqu’au moment où je me suis retrouvé bloqué et j’ai du passer python en 3.6 et django en 2.0. Après ça j’ai passé un peu de temps à réparer tout ce que je pouvais, avant d’avoir à nouveau des tests unitaires fonctionnels, et où j’ai pu continuer les mises à jour petit à petit.

Pour l’instant, tous les tests qui passaient quand j’ai commencé passent encore, sauf test_cant_change_username_when_editing que j’ai du skip, le temps qu’on mette à jour django-autocomplete-light en v3.

La CI a l’air contente aussi.

Mais je pense que ça nécessite encore un peu de travail à tester tout ça à la main pour voir ce que les tests unitaires ont raté, puis reprendre les mises à jour des différentes dépendances sur lesquelles j’ai mis quelques commentaires dans le requiremnts.txt, avant d’arriver à Bullseye / Python 3.9 pour pouvoir vraiment mettre à jour la prod.

PS: je suis à tetaneutral.net, et on n’utilise pas coin mais djadhere, donc j’ai pas trop d’expérience avec cette app, j’ai peut-être raté des trucs à cause de ça x)

Merge request reports