configglue team mailing list archive
-
configglue team
-
Mailing list archive
-
Message #00572
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