A PR that combines #1 ad #4

logstash-plugins / logstash-output-datadog_metrics

Apache License 2.0

3 stars 16 forks source link

A PR that combines #1 ad #4 #5

Closed guyboertje closed 9 years ago

guyboertje commented 9 years ago

Take the changes from PR 1 and PR 4, combine them, add tests and clean up.

jsvd commented 9 years ago

LGTM

purbon commented 9 years ago

Only left minor comments in the specs style, if so we can move forward and merge!

LGTM

jgmchan commented 9 years ago

:+1: It would be awesome to get this merged in as I would like to use the datadog_metrics output with Logstash 1.5. I've tested it using the fork and it's working great!

elasticsearch-bot commented 9 years ago

Merged sucessfully into master!