-
-
Notifications
You must be signed in to change notification settings - Fork 8.7k
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
Fixes #5843 #5849
Fixes #5843 #5849
Conversation
I'm awaiting a review of the CLA by my company. I will sign it when the review is complete. |
Thank you for your pull request! The most important CI builds succeeded, we’ll review the pull request soon. |
I have signed the CLA. |
@sokra it looks like automation has rejected the current change, but I'm not familiar enough with the state of the project (particularly: code coverage) to know how actionable this rejection is. Code coverage may have gone down due to the Can you advise? |
@sokra friendly ping on this. |
@sokra yet another friendly ping. |
The CI doesn't like to run correctly... |
Thanks |
What kind of change does this PR introduce?
This PR introduces a bugfix
Did you add tests for your changes?
Yes.
If relevant, link to documentation update:
N/A
Summary
Please refer to #5843 for a description of the bug and motivation for the change.
Does this PR introduce a breaking change?
No.