little tweak to our lgtm config file

[fchapoton]

so that files named all.py and catalog.py are ignored.

see

https://help.semmle.com/lgtm-enterprise/user/help/lgtm.yml-configuration-file.html#path_classifiers

CC: @embray @kiwifb @isuruf @tscrim

Component: refactoring

Author: Frédéric Chapoton

Branch/Commit: 967c510

Reviewer: Travis Scrimshaw

Issue created by migration from https://trac.sagemath.org/ticket/28995

[fchapoton]

New commits:

f75c866

little tweak to lgtm yaml config file

[fchapoton]

Branch: u/chapoton/28995

[fchapoton]

Commit: f75c866

[tscrim]

comment:4

How about excluding all files with catalog in the name?

[fchapoton]

comment:5

good idea, let me do that

[sagetrac-git]

Changed commit from f75c866 to 8c46cc1

[sagetrac-git]

Branch pushed to git repo; I updated commit sha1. New commits:

8c46cc1

still better lgtm.yml

[tscrim]

Reviewer: Travis Scrimshaw

[tscrim]

comment:7

Are there any catalog files of the form foo_catalog.py? If not, then positive review.

[sagetrac-git]

Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:

967c510

little tweak to lgtm yaml config file

[sagetrac-git]

Changed commit from 8c46cc1 to 967c510

[fchapoton]

comment:9

indeed, there are some..

[tscrim]

comment:10

Thanks. LGTM.

[vbraun]

Changed branch from u/chapoton/28995 to 967c510