Ignore by directory using Pylint

Question

The following is from the Pylint documentation:

--ignore=<file>
    Add <file or directory> to the black list. It should be a 
    base name, not a path. You may set this option multiple 
    times. [current: %default]

Yet, I'm not having luck getting the directory part work.

I have directory called migrations, which has django-south migration files. As I enter --ignore=migrations, it still keeps giving me the errors/warnings in files inside the migrations directory.

Could it be that --ignore is not working for directories?

If I could even use a regular expression to match the ignored files, it would work, since django-south files are all named 0001_something, 0002_something...

Since I could not get the ignore by directory to work, I have resorted to simply putting # pylint: disable-msg-cat=WCREFI on top of each migration file, which ignores all Pylint errors, warnings, and information.

pylint currently only respects the last `--ignore` option in your path name. Are you using `--ignore` multiple times maybe? http://www.logilab.org/ticket/22273 — badp, Mar 24 '10 at 08:05
Is this pydev-related? Perhaps add tags like pydev or eclipse? I'm searching for this as well. — michuelnik, Apr 05 '14 at 16:59
Does this answer your question? [Pylint: Disable specific warnings for specific folder](https://stackoverflow.com/questions/36182847/pylint-disable-specific-warnings-for-specific-folder) — rom_pep, Dec 18 '19 at 10:58

score 43 · Accepted Answer · edited Jan 17 '21 at 06:35

43

Adding the following to my .pylintrc files works with Pylint 0.25:

[MASTER]
ignore=migrations

My problems are with PyDev which (it seems) is not respecting my settings. This is due, I think, to the fact that it's running Pylint per-file, which I think bypasses 'ignore' checks - whether for modules/directories or files. The calls to Pylint from PyDev look like:

/path/to/site-packages/pylint/lint.py --include-ids=y /path/to/project/migrations/0018_migration.py

edited Jan 17 '21 at 06:35

Peter Mortensen

30,030
21
100
124

answered Dec 08 '11 at 06:40

marqueed

1,033
10
12

I can confirm that `ignore=migrations` still works with pylint 1.3.1 – Peterino Oct 21 '14 at 13:32
6

This did *not* work for me. However, using the **module-name** did the trick. So for a package like `foo/bar` I had to use `foo.bar` instead of `bar`. – exhuma Dec 21 '17 at 16:27

score 14 · Answer 2 · edited Jan 17 '21 at 06:37

14

To ignore subdirectories under a directory tree named 3rdparty, we added the following ignore-patterns entry to the [MASTER] entry in .pylintrc.

# Add files or directories matching the regex patterns to the blacklist. The
# regex matches against base names, not paths.
# Ignore all .py files under the 3rdparty subdirectory.
ignore-patterns=**/3rdparty/**/*.py

This fixed the problem for Pylint 1.7.1.

We were originally confused by the "base names" clause in the comments. Apparently it does accept paths with wildcards. At least it did for us.

edited Jan 17 '21 at 06:37

Peter Mortensen

30,030
21
100
124

answered Aug 31 '18 at 23:10

Joe Linoff

721
9
11

5

This does not seem to work for me (in pylint 1.9.2), first because it doesn't like the `**`, (I guess they changed it to the python regexp parser). Even after making it a proper regexp the `ignore-patterns` argument seems to only operate on the name of the file being tested, and does not seem to work for excluding a directory – Peter Aug 12 '19 at 15:07

score 12 · Answer 3 · edited Jan 17 '21 at 06:36

12

You can not give a path, but only the "basename" of the directory. E.g., use --ignore=lib instead of --ignore-=appengine-toolkit/gaetk/lib.

The problem is you will ignore all directories named lib.

edited Jan 17 '21 at 06:36

Peter Mortensen

30,030
21
100
124

answered Sep 09 '15 at 13:07

max

27,817
11
50
74

2

why is it like this??? why not ignore a whole directory?? I have to give all the file names or patterns for all the files in directory ??? this is not a correct way right – Lokesh Sanapalli Mar 04 '19 at 11:59

score 11 · Answer 4 · edited Jan 17 '21 at 06:42

As of right now, --ignore is not working on directories (there's an open issue on GitHub, Ignore clause not ignoring directories #2686).

It turns out that only the file name, and not the whole path is tested against the black list. (As pointed in the same pull request: geajack's comment)

The option --ignore-patterns has the same problem, but there was an attempt to fix it when I checked (Add ignore-paths to match against the full path #3266)

In your case, the best solution right now would be to use a regular expression to match the patterns you want in your files, which was also my case. As I am not really well-versed in regular expressions, I used Regex101, which I recommend.

score 3 · Answer 5 · edited Jan 17 '21 at 06:37

3

Actually, with Pylint 2.3.1 there is an open issue.

If you set a directory into the ignore options, it won't ignore it.

edited Jan 17 '21 at 06:37

Peter Mortensen

30,030
21
100
124

answered Oct 14 '19 at 12:18

Andrea Franchini

496
3
12

score 3 · Answer 6 · answered May 28 '21 at 09:04

Starting with Pylint 2.9, there will be a new configuration directory ignore-paths that matches a RegEx against the whole path and not the basename.

See also:

Jonathan Foster · Answer 7 · 2021-10-18T16:07:57.787

3

Adding to Fabien's answer, --ignore-paths syntax looks like this.

ignore-paths=^src/experiments/.*$,
             ^src/ingredients/.*$,
             ^src/scripts/.*$,
             ^src/tests/.*$

https://github.com/PyCQA/pylint/issues/2686#issuecomment-864585778

edited Oct 18 '21 at 16:07

answered Oct 18 '21 at 16:00

Jonathan Foster

73
1
6

score 1 · Answer 8 · edited Jan 17 '21 at 06:44

It seems you need to do some monkey patching, which works for me with Pylint version 2.5.3. I just don't know why Pylint doesn't have a fix for ignoring paths.

Add this to your .pylintrc file:

init-hook=
    sys.path.append(os.getcwd());
    from pylint_ignore import PylintIgnorePaths;
    PylintIgnorePaths('my/thirdparty/subdir', 'my/other/badcode')

Then create file pylint_ignore.py:

from pylint.utils import utils


class PylintIgnorePaths:
    def __init__(self, *paths):
        self.paths = paths
        self.original_expand_modules = utils.expand_modules
        utils.expand_modules = self.patched_expand

    def patched_expand(self, *args, **kwargs):
        result, errors = self.original_expand_modules(*args, **kwargs)

        def keep_item(item):
            if any(1 for path in self.paths if item['path'].startswith(path)):
                return False

            return True

        result = list(filter(keep_item, result))

        return result, errors

score -2 · Answer 9 · answered Mar 24 '10 at 07:57

-2

You can then use Bash expansion to your advantage:

--ignore=migrations/{0000..1000}_something

answered Mar 24 '10 at 07:57

badp

11,236
3
57
86

1

I think the ignore allowes only *basename*, so the `migrations/` would not work. Also I'm developing under Windows so I think Bash is out of question. – Ciantic Mar 24 '10 at 14:31
This post looks a little long so i'll ask again. Maybe something has changed. Is it possible to exclude the whole `migrations` directory? I'm on Windows as well. :( Thanks. – Mridang Agarwalla Oct 24 '11 at 14:21
1

Yes, this works for me. My line looks like: ignore=CVS,migrations – Harlin Mar 22 '17 at 03:45

Ignore by directory using Pylint

9 Answers9

Linked