Code review comment for lp:~nuclearbob/utah/consolidate-scripts

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

There still a few .rst files under docs/source/man that should be removed.

If you try to build the documentation you'll see some error messages because
those .rst files include the .txt file that was generated from the script help
output to generate the man page.

Regarding what should be the way to get a machine, I think a function or a
class method in the inventory module would be a good fit with how the code
works now.

review: Needs Fixing

« Back to merge proposal