-
-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Release v3.9.4 #8201
Release v3.9.4 #8201
Conversation
for more information, see https://pre-commit.ci
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## 3.9 #8201 +/- ##
=======================================
Coverage 97.39% 97.39%
=======================================
Files 108 108
Lines 32849 32849
Branches 3900 3900
=======================================
Hits 31994 31994
Misses 645 645
Partials 210 210
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
#8184 might be ready tomorrow but its not a regression and its been a problem for a while so no need to wait for it |
Probably a good one to have in, not sure if we want to do any more 3.9 releases after this. I'll do the release later in the week if you need time to finish it. |
I was hoping to get a thumbs up from the op on if it fixed it for his user before moving it forward. I'll write the tests for it tomorrow regardless, since I'll be time poor at the end of the week as I have family in town. |
@bdraco did you get that info? |
That one's merged now. We've had a couple of other last minute bug reports we've been fixing, but I think we're good for me to proceed with a release tomorrow. |
Sounds good! |
Preparing for tomorrow, let me know if there's any reason to delay. @bdraco @webknjaz