search

eric / metriks

An experimental library to instrument ruby
MIT License
478 stars 51 forks source link

Snapshot class pollutes top-level namespace #11

Closed brynary closed 12 years ago

brynary commented 12 years ago

Metriks has a Snapshot class which is not nested in the Metriks module. This conflicts with a Snapshot class I have in my app. Going to try and create a patch.

eric commented 12 years ago

Oh man, that's bad. Thanks for noticing.

brynary commented 12 years ago

Fixed in https://github.com/brynary/metriks

eric commented 12 years ago

Sweet. Would you mind making a pull request for it?

eric commented 12 years ago

I just grabbed it. Thanks for the commit.