deltacloud-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From David Lutterkort <lut...@redhat.com>
Subject Re: [PATCH core] Core: Code layout fixes and avoiding duplication
Date Tue, 11 Sep 2012 22:42:00 GMT
On Tue, 2012-09-11 at 14:49 +0200, mfojtik@redhat.com wrote:
> From: Michal Fojtik <mfojtik@redhat.com>
> 
> * Moved '[CIMI|Deltacloud]::Base' class to base_collection.rb.
>   Having this Sinatra class in 'helpers.rb' is not very intuitive.

Why not move it to {cimi,deltacloud}/collections/base.rb (rather than
base_collection.rb) ?

That's where I'd expect it to be given the class is
{Deltacloud,CIMI}::Collections::Base

David



Mime
View raw message