Development #36331
compat django 2.2
0%
Description
De l'avant.
Demandes liées
Révisions associées
misc: update decorated_includes() for new django patterns (#36331)
misc: update login/logout to be class based views (#36331)
tests: update API tests to work with latest django-webtest (#36331)
misc: add new required renderer argument to render() (#36331)
misc: remove usage of six (#36331)
misc: update from fractions.gcd to math.gcd (#36331)
tox: enable tests against django 2.2 (#36331)
misc: correctly reset current directory after translations (#36331)
tox: avoid django-mellon 1.6 (#36331)
tests: don't run migration tests with sqlite and newer django (#36331)
Historique
Mis à jour par Frédéric Péters il y a plus de 4 ans
- Statut changé de Nouveau à Solution proposée
Série de commits, dans la branche wip/36331-django-2.2, https://jenkins2.entrouvert.org/job/chrono-wip/job/wip%252F36331-django-2.2/ , dans l'ordre :
- 38dcb78 misc: add on_delete parameter to foreign keys (#36331)
C'est désormais obligatoire.
- f84f573 misc: get reverse{_lazy} from new django.urls module (#36331)
reverse et reverse_lazy doivent désormais être importés depuis django.urls.
- 617222e misc: update decorated_includes() for new django patterns (#36331)
Il y a eu du travail sur les pattern mais ça reste relativement clean et est en gros juste s/RegexURLPattern/URLPattern/, mais en passant je vire aussi RegexURLResolver qui pareil n'existe plus mais n'était de toute façon pas utilisé.
- 23e4fd1 misc: refer to user.is_anonymous as a property (#36331)
Trivial.
- 261bc99 misc: update MIDDLEWARE settings name (#36331)
Pareil trivial ici, et pour gérer ça côté hobo, #36335.
- 034d6b4 misc: update login/logout to be class based views (#36331)
Pas particulièrement différent.
- bf7ffaf tests: use .set() to set many-to-many value (#36331)
Désormais obligatoire.
- 48456f8 tests: update API tests to work with latest django-webtest (#36331)
Le dernier django-webtest vient avec sa propre classe d'authentification pour django rest framework ce qui fait que par défaut sur ces appels on récupère des 401 et plus des 403.
- 7c54c3f misc: add new required renderer argument to render() (#36331)
Sans commentaire.
- 9060dc3 misc: remove usage of six (#36331)
Ici on commence à virer la prise en charge en Python 2.
- d32cea5 misc: update from fractions.gcd to math.gcd (#36331)
Pareil.
- f51c6a9 tox: enable tests against django 2.2 (#36331)
Les tests pour 2.2.
- 7c9dd90 misc: correctly reset current directory after translations
Pas vraiment lié à tout le reste, juste un bug sur lequel je suis tombé à un moment où compilemessages échouait.
- db8617c tox: avoid django-mellon 1.6
C'est une release à éviter.
- d59be72 tests: don't run migration tests with sqlite and newer django
Il y a un truc de transaction/migration avec sqlite qui rend ces tests compliqués, après différentes tentatives de mocking de bouts genre enable_constraint_checking, plutôt je zappe. (cf https://code.djangoproject.com/ticket/30023)
Mis à jour par Benjamin Dauvergne il y a plus de 4 ans
- Statut changé de Solution proposée à Solution validée
Frédéric Péters a écrit :
- 38dcb78 misc: add on_delete parameter to foreign keys (#36331)
C'est désormais obligatoire.
ok.
- f84f573 misc: get reverse{_lazy} from new django.urls module (#36331)
reverse et reverse_lazy doivent désormais être importés depuis django.urls.
ok.
- 617222e misc: update decorated_includes() for new django patterns (#36331)
Il y a eu du travail sur les pattern mais ça reste relativement clean et est en gros juste s/RegexURLPattern/URLPattern/, mais en passant je vire aussi RegexURLResolver qui pareil n'existe plus mais n'était de toute façon pas utilisé.
ok.
- 23e4fd1 misc: refer to user.is_anonymous as a property (#36331)
Trivial.
ok.
- 261bc99 misc: update MIDDLEWARE settings name (#36331)
Pareil trivial ici, et pour gérer ça côté hobo, #36335.
Pour MIDDLEWARE il faut aussi le support dans debian_config_common.py, j'ai ouvert un ticket #36584.
- 034d6b4 misc: update login/logout to be class based views (#36331)
Pas particulièrement différent.
ok.
- bf7ffaf tests: use .set() to set many-to-many value (#36331)
Désormais obligatoire.
ok.
- 48456f8 tests: update API tests to work with latest django-webtest (#36331)
Le dernier django-webtest vient avec sa propre classe d'authentification pour django rest framework ce qui fait que par défaut sur ces appels on récupère des 401 et plus des 403.
ok.
- 7c54c3f misc: add new required renderer argument to render() (#36331)
Sans commentaire.
ok.
- 9060dc3 misc: remove usage of six (#36331)
Ici on commence à virer la prise en charge en Python 2.
ok.
- d32cea5 misc: update from fractions.gcd to math.gcd (#36331)
Pareil.
ok.
- f51c6a9 tox: enable tests against django 2.2 (#36331)
Les tests pour 2.2.
ok.
- 7c9dd90 misc: correctly reset current directory after translations
Pas vraiment lié à tout le reste, juste un bug sur lequel je suis tombé à un moment où compilemessages échouait.
ok.
- db8617c tox: avoid django-mellon 1.6
C'est une release à éviter.
ok.
- d59be72 tests: don't run migration tests with sqlite and newer django
Il y a un truc de transaction/migration avec sqlite qui rend ces tests compliqués, après différentes tentatives de mocking de bouts genre enable_constraint_checking, plutôt je zappe. (cf https://code.djangoproject.com/ticket/30023)
Il me semble qu'il faut utiliser transactional_db, dans une transaction sqlite je ne suis pas certain qu'on puisse faire tous les changements de schéma que l'on souhaite (en tout cas je ne pense pas que ce soit énormément testé coté Django1).
Je valide, juste attendre #36584.
1 https://code.djangoproject.com/ticket/29182, ce ticket est corrigé mais dans un commentaires ils pointent que lancer des migrations dans une transaction n'est jamais testé
Mis à jour par Benjamin Dauvergne il y a plus de 4 ans
- Bloqué par Development #36584: Support MIDDLEWARE nouveau style dans debian_config_common.py ajouté
Mis à jour par Frédéric Péters il y a plus de 4 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 5bc475c6634e735e58b8cb00d95045404ccd9e4f Author: Frédéric Péters <fpeters@entrouvert.com> Date: Sat Sep 21 21:20:08 2019 +0200 tests: don't run migration tests with sqlite and newer django (#36331) commit 95c09426c9afcc8c52c6c28c2480ba7ac802291f Author: Frédéric Péters <fpeters@entrouvert.com> Date: Sun Sep 22 13:09:07 2019 +0200 tox: avoid django-mellon 1.6 (#36331) commit 56b133c7a7aff380843e4dd8b9528e2755072886 Author: Frédéric Péters <fpeters@entrouvert.com> Date: Sun Sep 22 13:07:58 2019 +0200 misc: correctly reset current directory after translations (#36331) commit 9a31e83d1081e292dd6e72ed696113792753c5aa Author: Frédéric Péters <fpeters@entrouvert.com> Date: Sat Sep 21 21:22:27 2019 +0200 tox: enable tests against django 2.2 (#36331) commit 63bff03a87b4dcfc991d964162dc08f8d64fe031 Author: Frédéric Péters <fpeters@entrouvert.com> Date: Sat Sep 21 22:37:04 2019 +0200 misc: update from fractions.gcd to math.gcd (#36331) commit 4f1761a1a23e2346b9a8e0632a2f06d4edf9da28 Author: Frédéric Péters <fpeters@entrouvert.com> Date: Sat Sep 21 22:36:08 2019 +0200 misc: remove usage of six (#36331) commit 1db1486d5874ab67268f2adfa5a378ab790df42d Author: Frédéric Péters <fpeters@entrouvert.com> Date: Sat Sep 21 22:31:48 2019 +0200 misc: add new required renderer argument to render() (#36331) commit 3432648af790acc2c6ac051c136dff122a050ec6 Author: Frédéric Péters <fpeters@entrouvert.com> Date: Sat Sep 21 21:21:14 2019 +0200 tests: update API tests to work with latest django-webtest (#36331) commit fe7f3482c8a85048cb188a796195e5f52b055174 Author: Frédéric Péters <fpeters@entrouvert.com> Date: Sat Sep 21 21:19:33 2019 +0200 misc: update login/logout to be class based views (#36331) commit 7810f162650e488587a235d6686f00d3e0d962f3 Author: Frédéric Péters <fpeters@entrouvert.com> Date: Sat Sep 21 21:17:30 2019 +0200 misc: update decorated_includes() for new django patterns (#36331) commit bd6dbe50fd0777101ae2daea9c9129dfe7031726 Author: Frédéric Péters <fpeters@entrouvert.com> Date: Sat Sep 21 21:16:23 2019 +0200 misc: get reverse{_lazy} from new django.urls module (#36331)
Mis à jour par Frédéric Péters il y a plus de 4 ans
- Statut changé de Résolu (à déployer) à Solution déployée
Mis à jour par Frédéric Péters il y a plus de 3 ans
- Lié à Project management #49998: compatibilité django 2.2 ajouté
misc: get reverse{_lazy} from new django.urls module (#36331)