Page MenuHomePhabricator

tooling: add configuration file for lgtm.com service
ClosedPublic

Authored by stefan_schmidt on Nov 20 2019, 1:13 AM.

Details

Summary

It seems they run this on many Github projects automatically now:
https://lgtm.com/projects/g/Enlightenment/efl

From my playing around it had 970 "recommendations" to fix our FIXME
tags. Filtering out this noise by using this config file to exclude the
query.

It also seems to pick up the python scripts as Python2 and has wrong
reporting due to this as well.
Depends on D10698

Diff Detail

Repository
rEFL core/efl
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
stefan_schmidt created this revision.Nov 20 2019, 1:13 AM

It seems that this patch has no reviewers specified. If you are unsure who can review your patch, please check this wiki page and see if anyone can be added: https://phab.enlightenment.org/w/maintainers_reviewers/

stefan_schmidt requested review of this revision.Nov 20 2019, 1:13 AM
bu5hm4n accepted this revision.Nov 20 2019, 1:26 AM

cool to have that arround :)

This revision is now accepted and ready to land.Nov 20 2019, 1:26 AM
Closed by commit rEFL9f67ad59b0cf: tooling: add configuration file for lgtm.com service (authored by stefan_schmidt, committed by Marcel Hollerbach <mail@marcel-hollerbach.de>). · Explain WhyNov 20 2019, 1:31 AM
This revision was automatically updated to reflect the committed changes.
vtorri added a subscriber: vtorri.Nov 20 2019, 1:45 AM

btw, in code there are also TODO and XXX comments...