Fix 830 #461
Fix 830 #461
Conversation
|
🧙 Sourcery has finished reviewing your pull request! Tips
|
![sourcery-ai[bot]](https://avatars.githubusercontent.com/in/48477?s=60&v=4){kind=small}
There was a problem hiding this comment.
We've reviewed this pull request using the Sourcery rules engine. If you would also like our AI-powered code review then let us know.
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #461 +/- ##
==========================================
+ Coverage 95.72% 95.94% +0.22%
==========================================
Files 28 28
Lines 1639 1753 +114
==========================================
+ Hits 1569 1682 +113
- Misses 70 71 +1 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
We've reviewed this pull request using the Sourcery rules engine. If you would also like our AI-powered code review then let us know.
|
Please add some tests showing this fixing the same scenario from the issue you linked to. |
There was a problem hiding this comment.
We've reviewed this pull request using the Sourcery rules engine. If you would also like our AI-powered code review then let us know.
What kind of change does this PR introduce?
postgrest.exceptions.APIErrorsupabase-py#830The user who reported it already closed it (by using a workaround), but the fix still applies and improves the code.