Projet

Général

Profil

Development #33423

ne plus prendre en charge django 1.8

Ajouté par Frédéric Péters il y a presque 5 ans. Mis à jour il y a presque 5 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
27 mai 2019
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non


Fichiers

Révisions associées

Révision 496bdb52 (diff)
Ajouté par Frédéric Péters il y a presque 5 ans

general: stop supporting django 1.8 (#33423)

Historique

#1

Mis à jour par Frédéric Péters il y a presque 5 ans

#2

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.

#3

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.

#4

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.

#5

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.

#6

Mis à jour par Frédéric Péters il y a presque 5 ans

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).

#7

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/}
#8

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)
#9

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

Formats disponibles : Atom PDF