this is fine from an FFe POV but I have to say that I find this very
inelegant:

-            Optional('schema', default=VolumeSchema.gpt):
+            Optional('schema', default='gpt' if has_new_voluptuous()
+                     else VolumeSchema.gpt):

So now instead of using a (longer, more complex) symbolic name in place
of a string, we are using *both* a long, complex symbolic name *and* a
string.  That makes my teeth itch.  I would like to see us solve this
better.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1795610

Title:
  New voluptuous breaks ubuntu-image

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu-image/+bug/1795610/+subscriptions

-- 
ubuntu-bugs mailing list
[email protected]
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

Reply via email to