-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Issue 2616 #2617
Issue 2616 #2617
Conversation
Thank you @oleg-andreyev! Could you please fix the failing tests and add a changelog note for the provided fix? |
24c33e1
to
91c2919
Compare
@phansys please approve workflow. |
@oleg-andreyev, could you please fix the CI pipeline and add a changelog note? Thanks in advance. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #2617 +/- ##
==========================================
- Coverage 78.75% 78.39% -0.36%
==========================================
Files 163 167 +4
Lines 8593 8613 +20
==========================================
- Hits 6767 6752 -15
- Misses 1826 1861 +35 ☔ View full report in Codecov by Sentry. |
e7f3a3d
to
4dc0c71
Compare
Removed void
Co-authored-by: Javier Spagnoletti <[email protected]>
thanks @oleg-andreyev! |
fixed #2616