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.

Sign up for GitHub

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

Jump to bottom

Count the number of times that MetricPublishing.start() was called #23

Merged
merged 1 commit into from
Mar 5, 2018
Merged

Count the number of times that MetricPublishing.start() was called #23

merged 1 commit into from
Mar 5, 2018

Conversation

LakerBaker
Copy link
Contributor

to determine when it's time to actually shut down the metrics polling
when MetricPublishing.stop() is called.

Count the number of times that MetricPublishing.start() was called
902aef4 
to determine when it's time to actually shut down the metrics polling
when MetricPublishing.stop() is called.
@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 902aef4 on CountMetricPublishingStartCalls into 66d253b on master.

@bulicekj bulicekj merged commit 423984a into master Mar 5, 2018
@bulicekj bulicekj deleted the CountMetricPublishingStartCalls branch March 5, 2018 21:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@LakerBaker @coveralls @bulicekj