Skip to content

Commit

Permalink
Update djangorestframework (#135)
Browse files Browse the repository at this point in the history
* Upgrade djrf

* Remove silliness with two separate CI runs

Was previously there because DJRF required pytz

* Explicitly depend on pytz for dev env

To allow lint to work on all files, including ones that import pytz
  • Loading branch information
mfogel authored Jul 6, 2024
1 parent 5cc17cd commit b04a0dd
Show file tree
Hide file tree
Showing 3 changed files with 131 additions and 136 deletions.
12 changes: 1 addition & 11 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -155,17 +155,7 @@ jobs:
POSTGRES_DB: postgres
POSTGRES_USER: postgres
POSTGRES_PASSWORD: postgres
run: poetry run pytest --cov=timezone_field --ignore=tests/test_serializer_field.py

- name: Test serializer fiels with coverage
if: matrix.tz-engine == 'pytz'
env:
POSTGRES_HOST: localhost
POSTGRES_PORT: 5432
POSTGRES_DB: postgres
POSTGRES_USER: postgres
POSTGRES_PASSWORD: postgres
run: poetry run pytest --cov=timezone_field tests/test_serializer_field.py
run: poetry run pytest --cov=timezone_field

- name: Generate coverage report
run: poetry run coverage xml
Expand Down
Loading

0 comments on commit b04a0dd

Please sign in to comment.