configglue team mailing list archive

[Ricardo Kirkner <ricardo.kirkner@xxxxxxxxxxxxx>]

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

inischema was added before the python schema support was introduced and is really kept for backwards compatibility purposes only. The idea for inischema to remain is also that it's a more lightweight approach where you don't need to define a schema class for things to work (though that means you get less strict validation of types, and lose some other functionality compared to the Schema class).

> 
> 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/+merg
> e/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.