Veaceslav Mindru

Ack, will adjust. On Jan 25, 2018 12:08, "Will Thames" wrote: > As discussed elsewhere, I think if we augment ansible.cfg with an > [ansible-review] section, that should be relatively...

Hi :) doing fine I actually still do care, it just possible got off my radar.

I was thinking of how to implement this and started with smthing like. ``` 36 local_config = os.path.join(os.getcwd(), '.ansible-review', 'config.ini')$ 37 if os.path.isfile(ANSIBLE_CFG):$ 38 print("config found")$ 39 sys.exit(1)$ 40 if...

+1 this i am now having the same problem, though not sure this should go into setup.cfg i would rather have it under ~./.ansible-review/config.ini. @willthames what if we would have...

as alternative simply change the warn to info. ``` diff --git a/lib/ansiblereview/__main__.py b/lib/ansiblereview/__main__.py index c04f077..42b849d 100755 --- a/lib/ansiblereview/__main__.py +++ b/lib/ansiblereview/__main__.py @@ -94,5 +94,5 @@ def main(): info("Reviewing all of %s"...

@willthames let me know your opinion on this one, i Would go with change of the error to INFO it should not be a problem to have "alien" files in...

nope =( not a perms issue

I will try to install this now on my laptop. let's see how this goes.

same problem on Fedora 23

ok i got it partially working with ``` [vmindru@vmindru ttwatch]# git diff src/libttwatch.cpp diff --git a/src/libttwatch.cpp b/src/libttwatch.cpp index 985d2de..037914f 100644 --- a/src/libttwatch.cpp +++ b/src/libttwatch.cpp @@ -354,9 +354,9 @@ int ttwatch_open_device(libusb_device...