wip/75462-Mise-a-jour-django-upgrade-co-po #22

Manually merged
vdeniaud merged 8 commits from wip/75462-Mise-a-jour-django-upgrade-co-po into main 2023-03-16 10:53:56 +01:00
Owner
No description provided.
vdeniaud added 6 commits 2023-03-15 16:40:02 +01:00
vdeniaud force-pushed wip/75462-Mise-a-jour-django-upgrade-co-po from d93f9330aa to 2d35cc367b 2023-03-15 16:51:07 +01:00 Compare
vdeniaud force-pushed wip/75462-Mise-a-jour-django-upgrade-co-po from 2d35cc367b to 09568f3a71 2023-03-15 17:09:42 +01:00 Compare
vdeniaud force-pushed wip/75462-Mise-a-jour-django-upgrade-co-po from 09568f3a71 to 859f70f10d 2023-03-15 17:29:40 +01:00 Compare
vdeniaud force-pushed wip/75462-Mise-a-jour-django-upgrade-co-po from 859f70f10d to ae0db6c5be 2023-03-15 17:34:21 +01:00 Compare
vdeniaud changed title from WIP: wip/75462-Mise-a-jour-django-upgrade-co-po to wip/75462-Mise-a-jour-django-upgrade-co-po 2023-03-15 17:47:41 +01:00
pmarillonnet reviewed 2023-03-15 17:53:19 +01:00
@ -19,3 +19,3 @@
python3-chardet,
python3-cryptography (>= 1.3.4),
python3-django (>= 2:2.2),
python3-django (>= 3:3.2),
Owner

Attention ici ça va casser la possibilité de tirer django depuis bullseye-backports (qui est ce qu’on fait actuellement) : https://packages.debian.org/bullseye-backports/python3-django

Attention ici ça va casser la possibilité de tirer django depuis bullseye-backports (qui est ce qu’on fait actuellement) : https://packages.debian.org/bullseye-backports/python3-django
Author
Owner

Ah oui on dirait bien, je disais ici entrouvert/chrono#54 (comment) que j'avais fait ça sans comprendre et Benj avait l'air serein :)

Ah oui on dirait bien, je disais ici https://git.entrouvert.org/entrouvert/chrono/pulls/54#issuecomment-5607 que j'avais fait ça sans comprendre et Benj avait l'air serein :)
Owner

J'aurai du dire qu'il fallait vérifier l'historique des versions c'est vrai.

J'aurai du dire qu'il fallait vérifier l'historique des versions c'est vrai.
Author
Owner

Deux commits ici pour remettre dans les clous les AppConfig, car django-upgrade supprime les default_app_config désormais implicites (https://docs.djangoproject.com/en/3.2/releases/3.2/#automatic-appconfig-discovery) mais ça explose si on est ailleurs que dans apps.py.

Deux commits ici pour remettre dans les clous les AppConfig, car django-upgrade supprime les default_app_config désormais implicites (https://docs.djangoproject.com/en/3.2/releases/3.2/#automatic-appconfig-discovery) mais ça explose si on est ailleurs que dans apps.py.
bdauvergne approved these changes 2023-03-15 18:02:01 +01:00
Dismissed
tnoel dismissed bdauvergne’s review 2023-03-16 10:24:45 +01:00
Reason:

La dépendance debian sur python3-django n'est pas bonne, il faut indique >= 2:3.2 pour autoriser bullseye-backports

vdeniaud force-pushed wip/75462-Mise-a-jour-django-upgrade-co-po from ae0db6c5be to 8d08f3f194 2023-03-16 10:27:22 +01:00 Compare
vdeniaud requested review from tnoel 2023-03-16 10:28:26 +01:00
Author
Owner

(j'ai fait la modif, je reveux bien un ack par qui veut bien)

(j'ai fait la modif, je reveux bien un ack par qui veut bien)
Owner

Deux commits ici pour remettre dans les clous les AppConfig, car django-upgrade supprime les default_app_config désormais implicites (https://docs.djangoproject.com/en/3.2/releases/3.2/#automatic-appconfig-discovery) mais ça explose si on est ailleurs que dans apps.py.

Ok, vu. Relu le reste, c’est OK pour moi.

> Deux commits ici pour remettre dans les clous les AppConfig, car django-upgrade supprime les default_app_config désormais implicites (https://docs.djangoproject.com/en/3.2/releases/3.2/#automatic-appconfig-discovery) mais ça explose si on est ailleurs que dans apps.py. Ok, vu. Relu le reste, c’est OK pour moi.
pmarillonnet approved these changes 2023-03-16 10:52:13 +01:00
vdeniaud manually merged commit 8d08f3f194 into main 2023-03-16 10:53:56 +01:00
bdauvergne deleted branch wip/75462-Mise-a-jour-django-upgrade-co-po 2023-03-16 12:01:25 +01:00
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No Assignees
4 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: entrouvert/authentic#22
No description provided.