[PATCH 1/3] Start moving code from the controller to the logic

Pierre-Yves Chibon pingou at pingoured.fr
Wed Jan 23 12:57:24 UTC 2013


On Wed, 2013-01-23 at 07:52 -0500, Bohuslav Kabrda wrote:
> You probably forget to remove the committing from *Logic, right?

Oups, indeed.

> <snip>
> >      @classmethod
> > +    def add_coprs(cls, user, name, repos, selected_chroots):
> </snip>
> 
> I'd name this in singular form (add_copr).
> Otherwise it looks ok. If you agree, I'll commit and fix the tiny
> issues myself (or if you want, you can redo the patch again).

Sure, go ahead, I should work this afternoon :)

Pierre


More information about the copr-devel mailing list