← Back to team overview

cf-charmers team mailing list archive

Re: Refactored to use refactored charm-helpers (issue 91450050)

 

On 2014/05/19 19:20:32, cory.johns wrote:
> https://codereview.appspot.com/91450050/diff/20001/hooks/install
> File hooks/install (right):


https://codereview.appspot.com/91450050/diff/20001/hooks/install#newcode60
> hooks/install:60: common.FOG_CONNECTION,
> On 2014/05/19 18:52:05, benjamin.saller wrote:
> > As for config/common naming, I don't mind either as long as we keep
it
> > consistent.

> The reason we renamed it to common was that it had the shared
db_migrate method
> in it.  With the removal of the partial sqlite support, that could be
moved into
> hooks.py and the impetus for naming it common is moot.  So perhaps I
will change
> it back to be consistent with the other charms.

Sounds good, thanks

https://codereview.appspot.com/91450050/

-- 
https://code.launchpad.net/~johnsca/charms/trusty/cf-cloud-controller/refactor/+merge/219919
Your team Cloud Foundry Charmers is requested to review the proposed merge of lp:~johnsca/charms/trusty/cf-cloud-controller/refactor into lp:~cf-charmers/charms/trusty/cf-cloud-controller/trunk.


References