-
-
Notifications
You must be signed in to change notification settings - Fork 8
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Resolved pytest errors by correctly annotating ORM relationships with Mapped type #108
Conversation
@peterdudfield , this PR will close #88 |
@peterdudfield , i think we need to update sqlalchemy version here https://github.com/openclimatefix/pv-site-datamodel/blob/main/poetry.lock#L16 to enable it on pv-site-app. To fix openclimatefix/pv-site-api#133 |
Looks like we still get tests failing - https://github.com/openclimatefix/pv-site-datamodel/actions/runs/8350094672/job/22855747461 |
can you run pytest on this repo? |
@peterdudfield, I checked |
see #110 |
@peterdudfield , let me check. |
If you make a new PR, the tests should automatically run, so you'll be able to see whats going on |
* updated version of sqlalchemy (#107) * Resolved pytest errors by correctly annotating ORM relationships with Mapped type (#108) * updated version of sqlalchemy * fixed the failing of pytest because of sqlalchemy --------- Co-authored-by: Peter Dudfield <34686298+peterdudfield@users.noreply.github.com> * lint * fix --------- Co-authored-by: Rosheen Naeem <rosheennaeem4@gmail.com>
Pull Request
Description
Fixed pytests failing because of sqlalchemy, see also #107
Fixes #88