← Back to team overview

configglue team mailing list archive

Re: [Merge] lp:~jzhuge/configglue/short_name_for_inischema into lp:configglue

 

Sure.

BTW, how do you compare inischema with Schema class? What is the use
case for inischema? Shall we make App work with inischema?

Thanks,

On Tue, Oct 11, 2016 at 4:37 AM, Ricardo Kirkner
<ricardo.kirkner@xxxxxxxxxxxxx> wrote:
> Hi,
>
> thanks for your merge proposal. Code looks good, but could you also add a matching test for it?
> --
> https://code.launchpad.net/~jzhuge/configglue/short_name_for_inischema/+merge/306699
> You are the owner of lp:~jzhuge/configglue/short_name_for_inischema.



-- 
John

https://code.launchpad.net/~jzhuge/configglue/short_name_for_inischema/+merge/306699
Your team Configglue developers is requested to review the proposed merge of lp:~jzhuge/configglue/short_name_for_inischema into lp:configglue.


Follow ups

References