LGTM (looks good to me) is a static code analyzer for Open Source projects that can analyse Python, JS, and other code. Static code analysis can find errors in code without actually executing it. It's very useful to find bugs in lines that are rarely executed.
https://lgtm.com/projects/g/freeipa/freeipa/ lists about 120 open errors and 80 open warnings for FreeIPA. Most errors look legit and should be addressed.
First patch: https://github.com/freeipa/freeipa/pull/1429
Metadata Update from @cheimes: - Issue assigned to cheimes
master:
ipa-4-6:
The first batch of fixes closed about 90 warnings and errors. FreeIPA is now down to 54 errors and 78 warnings.
Metadata Update from @rcritten: - Issue priority set to: normal - Issue set to the milestone: FreeIPA 4.7
Metadata Update from @rcritten: - Issue set to the milestone: FreeIPA 4.7.1 (was: FreeIPA 4.7)
FreeIPA 4.7 has been released, moving to FreeIPA 4.7.1 milestone
Metadata Update from @rcritten: - Issue set to the milestone: FreeIPA 4.7.2 (was: FreeIPA 4.7.1)
FreeIPA 4.7.1 has been released, moving to FreeIPA 4.7.2 milestone
There are still a few warnings/suggestions. I skimmed them and most were bogus but probably worth a closer look.
Login to comment on this ticket.