Re: [PATCH 4/4] Move the URL to copr into the config file
On Mon, Feb 18, 2013 at 01:53:53AM -0500, Bohuslav Kabrda wrote:
All four patches look fine to me, please go ahead and commit (if you
were waiting for my review, please don't wait next time - at least not with copr_cli
related stuff, I'd only like to review patches that go into the frontend itself).
Noted, but I was hoping to get some feed-back on the approach I took (using
argparse, the way I handle subcommands & all).
Also, now that I think about it, I realize I need to tweak some things before
pushing all this, I do not provide the same user experience with these patches
as we have now, I need to fix this first.
Thanks for the feedback,
Pierre