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

Export the flush action so that it can be invoked #18

Merged
merged 2 commits into from
Mar 23, 2018

Conversation

dfithian
Copy link
Contributor

We have been using this library for some time, love exporting metrics to statsd (using the datadog statsd backend). However, we have never been able to use this in conjunction with a script as the statsd flusher can't be invoked manually. Hopefully this change is accepted; I have tested it using the script in question and it works great.

Copy link
Collaborator

@23Skidoo 23Skidoo left a comment

Choose a reason for hiding this comment

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

LGTM.

@dfithian
Copy link
Contributor Author

Thanks for review! Trying to get my stack.yamls back in line... is there a typical time for merging and releasing? (happy to do it myself as well)

@23Skidoo 23Skidoo merged commit 67464a0 into haskell-github-trust:master Mar 23, 2018
@23Skidoo
Copy link
Collaborator

Merged, thanks! I'll make a release later this weekend.

@23Skidoo 23Skidoo self-assigned this Mar 25, 2018
@23Skidoo
Copy link
Collaborator

Don't have time for this today, will do at some point during the week.

@dfithian
Copy link
Contributor Author

Okay, thank you for your time!

@dfithian
Copy link
Contributor Author

Hi, any possibility to do this soonish? Sorry to bug. Should really stop being a polyrepo company 🤦‍♂️ .

@23Skidoo
Copy link
Collaborator

0.2.3.0 released, sorry for the long wait.

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