Re: [libvirt] [jenkins-ci PATCH v2 04/12] lcitool: Add inventory handling
On Tue, 2018-07-17 at 11:57 +0200, Katerina Koukiou wrote:
On Thu, Jul 12, 2018 at 05:19:21PM +0200, Andrea Bolognani wrote:
> +import yaml
Since you have to install yaml to actually use it, I suggest at this
point to create a requirements.txt file for this script, so that users
don't have to go over the code to check the dependencies.
Neat idea.
If that's okay with you, I'd rather work on that as a follow-up
enhancement though, because I need to do some reading on it and
I'd rather not leave this series lingering for too long unless
it's really necessary.
--
Andrea Bolognani / Red Hat / Virtualization