Skip to content
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

fix(client): flush resultsBuffer on engine upgrade #3212

Merged
merged 1 commit into from
Apr 29, 2020
Merged

fix(client): flush resultsBuffer on engine upgrade #3212

merged 1 commit into from
Apr 29, 2020

Conversation

tbhartman
Copy link
Contributor

Fixes #3211

@googlebot
Copy link

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

📝 Please visit https://cla.developers.google.com/ to sign.

Once you've signed (or fixed any issues), please reply here (e.g. I signed it!) and we'll verify it.


What to do if you already signed the CLA

Individual signers
Corporate signers

@tbhartman
Copy link
Contributor Author

@googlebot I signed it!

@googlebot
Copy link

CLAs look good, thanks!

@karmarunnerbot
Copy link
Member

Build karma 220 failed (commit e47af6bb53 by @tbhartman)

@AppVeyorBot
Copy link

Build karma 2618 failed (commit e47af6bb53 by @tbhartman)

@karmarunnerbot
Copy link
Member

Build karma 219 failed (commit e47af6bb53 by @tbhartman)

@googlebot
Copy link

All (the pull request submitter and all commit authors) CLAs are signed, but one or more commits were authored or co-authored by someone other than the pull request submitter.

We need to confirm that all authors are ok with their commits being contributed to this project. Please have them confirm that by leaving a comment that contains only @googlebot I consent. in this pull request.

Note to project maintainer: There may be cases where the author cannot leave a comment, or the comment is not properly detected as consent. In those cases, you can manually confirm consent of the commit author(s), and set the cla label to yes (if enabled on your project).

ℹ️ Googlers: Go here for more info.

@AppVeyorBot
Copy link

Build karma 2619 completed (commit d012a9f285 by @devoto13)

@karmarunnerbot
Copy link
Member

Build karma 221 completed (commit d012a9f285 by @devoto13)

@googlebot
Copy link

CLAs look good, thanks!

ℹ️ Googlers: Go here for more info.

@karmarunnerbot
Copy link
Member

Build karma 220 completed (commit d012a9f285 by @devoto13)

@AppVeyorBot
Copy link

Build karma 2620 completed (commit 51435215c1 by @tbhartman)

@karmarunnerbot
Copy link
Member

Build karma 222 completed (commit 51435215c1 by @tbhartman)

Copy link
Collaborator

@devoto13 devoto13 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

I've added a check to make broken tests pass. The fix for that issue is available in #3486.

The reasoning for the fix is in #3211 (comment).

@johnjbarton I think this PR should be good to merge now.

@karmarunnerbot
Copy link
Member

Build karma 221 completed (commit 51435215c1 by @tbhartman)

Copy link
Contributor

@johnjbarton johnjbarton left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The test for this change is in #3487

@johnjbarton johnjbarton merged commit e44ca94 into karma-runner:master Apr 29, 2020
karmarunnerbot pushed a commit that referenced this pull request Apr 29, 2020
## [5.0.3](v5.0.2...v5.0.3) (2020-04-29)

### Bug Fixes

* **client:** flush resultsBuffer on engine upgrade ([#3212](#3212)) ([e44ca94](e44ca94)), closes [#3211](#3211)
@karmarunnerbot
Copy link
Member

🎉 This PR is included in version 5.0.3 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Reporter writes spec results out of order
6 participants