codecov[bot]
commented
1 year ago Codecov Report
Merging #439 (074343e) into master (c6428f9) will decrease coverage by
0.13%. The diff coverage isn/a.:exclamation: Current head 074343e differs from pull request most recent head 9388f2f. Consider uploading reports for the commit 9388f2f to get more accurate results
@@ Coverage Diff @@
## master #439 +/- ##
==========================================
- Coverage 95.72% 95.58% -0.14%
==========================================
Files 76 76
Lines 4418 4418
Branches 470 656 +186
==========================================
- Hits 4229 4223 -6
- Misses 141 143 +2
- Partials 48 52 +4 | Impacted Files | Coverage Δ | |
|---|---|---|
| weblab/fitting/forms.py | 90.47% <0.00%> (-6.35%) |
:arrow_down: |
| weblab/entities/views.py | 92.61% <0.00%> (-0.28%) |
:arrow_down: |
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more
So I've just been messing about to see if we can support python 3.11. Apart of some of the versions that were pinned causing problems, the main issue seems to be psycopg2 the older version (psycopg2==2.8.6) doesn't seem to work with python 3.11 and the newer version throws errors when using timezones is set on.
It's a simple fix to switch that off, but I'm not sure if that's such a great idea. It also means comparing commits & version timestamps need to compare ignoting timezone. I don't know whether this would be ok in weblab and if it's a good idea @jonc125
I suspect ultimately it would want a django update, as we're on 2.2.28, while 4.1.3 is now the latest.