Conversation
|
Warnings, unit-py313-django52: Regexp warnings: Other warnings: |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #3467 +/- ##
==========================================
+ Coverage 62.05% 62.07% +0.02%
==========================================
Files 611 611
Lines 44854 44854
Branches 43 43
==========================================
+ Hits 27835 27845 +10
+ Misses 17009 16999 -10
Partials 10 10 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Fixed by upgrading feedparser. |
|
Warnings, integration-py313-django52: I don't think there's anything here fixable by us. |
|
functional-py313-django52 has no additional warnings or errors, green! |
This is an explicit NAV warning. You will almost always see this in a dev or test environment where the database hasn't been configured (or in production if the |
d2d2aaf to
9082e1b
Compare
johannaengland
left a comment
There was a problem hiding this comment.
Just small things
5632622 to
2696aaa
Compare
2696aaa to
6af3c50
Compare
|



Scope and purpose
Dependent on #3266.
Django 4.2 does not support python 3.13 so we need to upgrade Django for the sake of running on debian trixie.
Replaces #3269, but see that for a list of usage of naive datetimes.
Looks of dates and numbers might change, we might have to manually look through the frontend and override if necessary.
This pull request
We need a newer feedparser and psycopg2. Psycopg2 is in maintenance mode, we should consider switching to psycopg 3.
Contributor Checklist
Every pull request should have this checklist filled out, no matter how small it is.
More information about contributing to NAV can be found in the
Hacker's guide to NAV.
<major>.<minor>.x). For a new feature or other additions, it should be based onmaster.[ ] If this results in changes in the UI: Added screenshots of the before and after[ ] If this adds a new Python source code file: Added the boilerplate header to that file