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

Coverband#start kicks off background thread #139

Merged
merged 2 commits into from
Nov 13, 2018

Conversation

kbaum
Copy link
Collaborator

@kbaum kbaum commented Nov 12, 2018

Unless running in rack server. In this case middleware kicks off on first request.

This PR is an addition to #138

Unless running in rack server. In this case middleware kicks off on
first request.
@kbaum kbaum requested a review from danmayer November 12, 2018 23:47
@kbaum kbaum force-pushed the background_thread_reporting_from_start branch from 20bffbe to 42568ea Compare November 12, 2018 23:51
Copy link
Owner

@danmayer danmayer left a comment

Choose a reason for hiding this comment

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

nice this looks great thanks @kbaum

Copy link
Owner

@danmayer danmayer left a comment

Choose a reason for hiding this comment

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

feel free to merge when ready.

@kbaum kbaum merged commit 3165c38 into master Nov 13, 2018
@kbaum
Copy link
Collaborator Author

kbaum commented Nov 13, 2018 via email

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

Successfully merging this pull request may close these issues.

2 participants