inspektor: Handle incorrect locations

avocado-framework / inspektor

Inspektor code checker

Other

11 stars 17 forks source link

Closed ldoktor closed 8 years ago

ldoktor commented 8 years ago

When checking incorrect location, the check() returns None without any explanation. This patch logs an info and returns False as the check failed.

Signed-off-by: Lukáš Doktor ldoktor@redhat.com

ldoktor commented 8 years ago

This is an improved version of https://github.com/autotest/inspektor/pull/17 @tripples can you confirm it works for you?

lmr commented 8 years ago

It tests OK as well:

scripts/inspekt style sbrubles
PEP8 disabled: E501,E265,W601,E402
Invalid location 'sbrubles'
PEP8 compliance FAIL

lmr commented 8 years ago

Thanks @ldoktor !