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