search

avocado-framework / inspektor

Inspektor code checker
Other
11 stars 17 forks source link

Port to cliff #26

Closed lmr closed 7 years ago

lmr commented 7 years ago

This PR makes inspektor to use the openstack originated cliff framework to build command line applications:

https://docs.openstack.org/cliff/latest/

As well as some python3 support bugfixes

lmr commented 7 years ago

To my reviewers:

I'll probably need some help fixing the .spec file properly. Let me know in what distros inspektor is being packaged for. I managed to get things working on py2.6, but that's really a lousy target that I'm keeping around just in case someone needs it.

clebergnu commented 7 years ago

@lmr this looks like a nice set of changes. I'm queueing the review in my TODO list here.

lmr commented 7 years ago

@ldoktor I implemented your suggestions and added some changes, when you have time let me know what you think. Thanks!

ldoktor commented 7 years ago

Hello Lucas, yes, I'm sorry for neglecting this for 18 days, it's been rough. I'll give it a try and send fixes if necessary (anyway it was quite fine). After some stabilization period we can release it, but I'd wait a while...

lmr commented 7 years ago

@ldoktor No problem man, I know you guys are busy. I have a few other plans, but I wanted to get this first step out of the door.