signalfx / signalfx-go

Go client library and instrumentation bindings for SignalFx
https://www.signalfx.com
Apache License 2.0
14 stars 48 forks source link

Add type for rule notifications #48

Closed cory-signalfx closed 5 years ago

cory-signalfx commented 5 years ago

Summary

Add type to detector rule notifications

Motivation

At the time this was spun up, I didn't invest the time to add types for all the notifications and cheated by using a []map[string]interface{}. The code from #25 gave us these types and #46 put them in a sane place to avoid cycling deps. Yay for type checking!