launchpad-reviewers team mailing list archive
-
launchpad-reviewers team
-
Mailing list archive
-
Message #06440
Re: [Merge] lp:~rvb/maas/config-table into lp:maas
> Moreover, the combination of "store lists as config values" and "store config
> values and I'll turn them into a list" seems destined to cause confusion, much
> like the trouble we had with Django's weird request parameter dicts. What
> exactly is the overriding requirement? Without one, I would say please keep
> it simple. The need for this kind of thing would be useful to explain in your
> cover letter, i.e. the description on the MP. But it's hard for me to judge
> without knowing more about the problem this branch solves.
I'm currently working on the next branch that uses these methods.
--
https://code.launchpad.net/~rvb/maas/config-table/+merge/93861
Your team Launchpad code reviewers is requested to review the proposed merge of lp:~rvb/maas/config-table into lp:maas.
References