Allow for easy instrumentation

[jnunemaker]

No description provided.

[technoweenie]

I don't know what you're planning, but ActiveSupport::Notifications is pretty good. I have a good backport that we use on .com so you don't necessarily need all of AS. https://github.com/technoweenie/activesupport_notifications_backport

[jnunemaker]

Yeah I just added instrumentation to flipper and ill probably do it the same way. It just requires the interface to be right and not necessarily AS.

On Jan 19, 2013, at 4:28 PM, rick notifications@github.com wrote:

I don't know what you're planning, but ActiveSupport::Notifications is pretty good. I have a good backport that we use on .com so you don't necessarily need all of AS. https://github.com/technoweenie/activesupport_notifications_backport

—
Reply to this email directly or view it on GitHub.

[technoweenie]

Oh right on. I've been doing the same in Faraday.

On Sat, Jan 19, 2013 at 3:54 PM, John Nunemaker notifications@github.comwrote:

Yeah I just added instrumentation to flipper and ill probably do it the
same way. It just requires the interface to be right and not necessarily
AS.

On Jan 19, 2013, at 4:28 PM, rick notifications@github.com wrote:

I don't know what you're planning, but ActiveSupport::Notifications is
pretty good. I have a good backport that we use on .com so you don't
necessarily need all of AS.
https://github.com/technoweenie/activesupport_notifications_backport

—
Reply to this email directly or view it on GitHub.

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/4#issuecomment-12463446.

Rick Olson
http://github.com/technoweenie

[jnunemaker]

Sweet. I'll play around with this in the coming week. It was really easy to add to flipper.

flippercloud/flipper#15

[jnunemaker]

Closed thanks to #9