-
Notifications
You must be signed in to change notification settings - Fork 1.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
chore(test): add test for flush resultsBuffer on engine upgrade #3487
Conversation
✅ Build karma 223 completed (commit a119c05bd4 by @devoto13) |
✅ Build karma 224 completed (commit a119c05bd4 by @devoto13) |
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.
Thanks!
Actually I see we are blocked by #3486. Could that change have unintended side-effects? |
65a01d3
to
3dc949d
Compare
✅ Build karma 227 completed (commit 37bfb42312 by @devoto13) |
✅ Build karma 226 completed (commit 37bfb42312 by @devoto13) |
🎉 This PR is included in version 5.0.4 🎉 The release is available on: Your semantic-release bot 📦🚀 |
A test case to catch #3212 regressions. It fails now, but will pass once the other PR is merged and test is rebased.