klaustopher/cloudmade — pull request #1 by arussell (2012-04-23 20:38:25+00:00)
Title: Avoid global namespace pollution

Seems this gem pollutes the global namespace so that the various class names (eg "Location") can't be used by our Rails app. Not being able to use certain class names is kind of a problem when building a location-aware Rails app that contains objects such as MyApp::Locations (as opposed to CloudMade::Locations).

This seems to be caused by a bunch of "include CloudMade" statements throughout the gem's source. I've removed these statements from various places in the source to avoid polluting global namespace.

I thought I may need to add "CloudMade::" in a couple places to counteract this, but everything in lib/cloudmade/example.rb seems to run okay without it (likely because it's all wrapped in "Module CloudMade...end"?), and it's no longer polluting the global namespace for Rails apps that have the gem in their Gemfile.

I bumped the version too, just to differentiate it from the version already on RubyGems.org. Not sure how you wanted to handle versioning so I just bumped the incremental.