search

OpenLightingProject / ola

The Open Lighting Architecture - The Travel Adaptor for the Lighting Industry
https://www.openlighting.org/ola/
Other
640 stars 202 forks source link

Consider using codacy #1202

Open peternewman opened 7 years ago

peternewman commented 7 years ago

https://www.codacy.com/

janosvitok commented 7 years ago

Codacy uses cppcheck (#1083) for C++ checks, but it seems it's not easy to set library in codacy (QLC+ should use library=qt, for OLA it may not matter).

peternewman commented 7 years ago

Thanks for the heads up @janosvitok . I clicked the magic button and it installed itself into GitHub yesterday.

It seems to have completed it's scan, but it's generating what looks like a lot of false positives related to it not knowing about the cppunit library and others, which I guess is similar to what you mentioned.

It seems to have generated a lot of low priority or seemingly false positive alerts, and a handful of genuine ones, so it feels like it's currently got too much noise for me, but perhaps it needs revisiting, or native cppcheck should be tried.