entrouvert
/
welco
17
0
Fork 0
Code Issues Pull Requests 2 Releases Activity

fonctionner avec djangorestframework 3.14 (#81950) #5

Merged
fpeters merged 2 commits from wip/81950-drf-314 into main 2023-10-05 09:53:19 +02:00
Conversation 1 Commits 2 Files Changed 2 +3 -6
2 changed files with 3 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
setup.py
View File

@ -113,7 +113,7 @@ setup(
'django-ckeditor<4.5.4',
'django-haystack<3.2',
'django-taggit<3',
'djangorestframework>=3.3,<3.13',
'djangorestframework>=3.3,<3.15',
'requests',
'whoosh',
'XStatic-Select2',

7
tox.ini
View File

@ -1,7 +1,6 @@
[tox]
envlist =
py3-django22-drf39
py3-django22-drf312
py3-django32-drf314
py3-django32-black-coverage-pylint-drf312
tnoel marked this conversation as resolved
tnoel commented 2023-10-03 16:58:28 +02:00
Review
Copy Link

Passer ici aussi en drf314 ?

Passer ici aussi en drf314 ?
fpeters commented 2023-10-03 17:01:51 +02:00
Review
Copy Link

L'idée était de continuer sur drf312 quand même, parce que c'est ça qu'on a sur les prods actuelles.

L'idée était de continuer sur drf312 quand même, parce que c'est ça qu'on a sur les prods actuelles.
fpeters commented 2023-10-03 17:05:34 +02:00
Review
Copy Link

L'idée était de continuer sur drf312 quand même, parce que c'est ça qu'on a sur les prods actuelles.

À noter qu'ici parce que c'est welco et on ne développe pas dessus et que jusqu'au commit précédent les tests tournaient déjà trois fois. Mais dans les autres modules, j'ai zéro envie de multiplier par deux le temps passé dans jenkins, donc mon idée sera de lancer la PR avec un premier commit sur le setup.py pour monter la limite, pour voir que les tests tournent, puis un autre commit pour mettre une limite <3.13 au niveau du tox.ini, pour continuer à builder sur cette version bullseye uniquement. (et quand s'approchera le moment de monter de version debian, monter ici).

> L'idée était de continuer sur drf312 quand même, parce que c'est ça qu'on a sur les prods actuelles. À noter qu'ici parce que c'est welco et on ne développe pas dessus et que jusqu'au commit précédent les tests tournaient déjà trois fois. Mais dans les autres modules, j'ai zéro envie de multiplier par deux le temps passé dans jenkins, donc mon idée sera de lancer la PR avec un premier commit sur le setup.py pour monter la limite, pour voir que les tests tournent, puis un autre commit pour mettre une limite <3.13 au niveau du tox.ini, pour continuer à builder sur cette version bullseye uniquement. (et quand s'approchera le moment de monter de version debian, monter ici).
toxworkdir = {env:TMPDIR:/tmp}/tox-{env:USER}/welco/{env:BRANCH_NAME:}
@ -15,9 +14,7 @@ setenv =
fast: FAST=--nomigrations
coverage: COVERAGE=--junitxml=junit-{envname}.xml --cov-report xml --cov-report html --cov=welco/
deps =
django22: django>=2.2,<2.3
django32: django>=3.2,<3.3
django22: django-haystack<2.8
pytest-cov
pytest-django
pytest!=5.3.3
@ -33,8 +30,8 @@ deps =
lxml
git+https://git.entrouvert.org/debian/django-ckeditor.git
black: pre-commit
drf39: djangorestframework>=3.9,<3.10
drf312: djangorestframework>=3.12,<3.13
drf314: djangorestframework>=3.14,<3.15
commands =
tnoel commented 2023-10-03 16:58:55 +02:00
Outdated
Review
Copy Link

Ca doit être supprimable, comme le 312 en dessous ?

Ca doit être supprimable, comme le 312 en dessous ?
fpeters commented 2023-10-03 17:02:12 +02:00
Outdated
Review
Copy Link

Et les dj22 etc. aussi.

Et les dj22 etc. aussi.
pylint: ./pylint.sh welco/
py.test {env:COVERAGE:} {posargs:tests/}