Code review comment for lp:~nuclearbob/utah/config-changes

Revision history for this message
Javier Collado (javier.collado) wrote :

It would be really nice to have some documentation in config.py for each configuration variable.
Looking at them, isn't really obvious what they're supposed to be used for.

I also agree with Joe's comment. However, since uqt-vm-tools.conf isn't a python file, we can still keep that as it is for now. Adding a comment wherever SERIES_CHOICES is defined to remind us to keep vm_release_list in sync would be nice to have though.

« Back to merge proposal