Development #33423
ne plus prendre en charge django 1.8
0%
Fichiers
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a presque 5 ans
- Fichier 0001-general-stop-supporting-django-1.8-33423.patch 0001-general-stop-supporting-django-1.8-33423.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Emmanuel Cazenave il y a presque 5 ans
Il faudrait :
py.test {env:COVERAGE:} {posargs:tests/} │ - ./pylint.sh combo/ │ + coverage-py2-pylint: ./pylint.sh combo/
sinon pylint va tourner deux fois (c'est un peu long).
Mais surtout, en passant --junitxml=test_results.xml
dans coverage, on a plus de fichiers junit généré pour py3.
Mis à jour par Emmanuel Cazenave il y a presque 5 ans
Emmanuel Cazenave a écrit :
sinon pylint va tourner deux fois (c'est un peu long).
Je vérifie ça j'ai peut-être dit une bêtise.
Mis à jour par Frédéric Péters il y a presque 5 ans
Pour référence j'ai tiré en grande partie ça de ce qu'on fait dans chrono, qui est un module proche, passé à django 1.11 et ok avec Python 3; comme différences où je suis resté conservateur dans combo, il y a justement l'exécution du coverage en python 3, qu'on ne déjà faisait pas.
Mis à jour par Emmanuel Cazenave il y a presque 5 ans
Je confirme ce que je disais.
Dans chrono il y a envlist = py2-coverage-pylint,py3-coverage
qui assure qu'on fait le coverage sur les deux envs et qu'on a les fichiers junits les deux coups aussi.
Idem il y a pylint: ./pylint.sh chrono/
qui assure que pylint tourne une seule fois, sur py2-coverage-pylint.
Mis à jour par Frédéric Péters il y a presque 5 ans
- Fichier 0001-general-stop-supporting-django-1.8-33423.patch 0001-general-stop-supporting-django-1.8-33423.patch ajouté
Pour le coverage, je ne veux pas faire tourner celui en python 3 (cf note précédente), qui sera incomplet tant que personne ne regarde à #33244. J'ai modifié pour ne pas exécuter pylint (même si je ne pense pas que ça soit bien long, ça ne sert à rien).
Mis à jour par Emmanuel Cazenave il y a presque 5 ans
Frédéric Péters a écrit :
Pour le coverage, je ne veux pas faire tourner celui en python 3.
Si tu veux, pas de soucis avec ça, je veux juste qu'on ait les résultats des tests unitaires en python3 :
- coverage: COVERAGE=--junitxml=test_results.xml --cov-report xml --cov-report html --cov=combo/ + coverage: COVERAGE=--cov-report xml --cov-report html --cov=combo/ deps = django>=1.11,<1.12 pywebpush @@ -32,5 +32,5 @@ commands = ./getlasso.sh python manage.py compilemessages ./get_wcs.sh - py.test {env:COVERAGE:} {posargs:tests/} + py.test {env:COVERAGE:} {posargs: --junitxml=test_{envname}_results.xml tests/}
Mis à jour par Frédéric Péters il y a presque 5 ans
- Statut changé de Solution proposée à Résolu (à déployer)
Ok, changé avec ça,
commit 496bdb525f67bf87abe78f31178810e0d8e91d75 Author: Frédéric Péters <fpeters@entrouvert.com> Date: Mon May 27 10:19:32 2019 +0200 general: stop supporting django 1.8 (#33423)
Mis à jour par Frédéric Péters il y a presque 5 ans
- Statut changé de Résolu (à déployer) à Solution déployée
general: stop supporting django 1.8 (#33423)