[tooling] adds non-hailtop subdirs to pylint ignore #14060
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
We only have
make
commands for runningpylint
on subdirectories that have been kept up to date with its rules, but thepylintrc
doesn't actually contain any indication of which subdirectories should be ignored when runningpylint
. This makes the use of language servers that runpylint
on the file that's open frustrating, as files in the ignored subdirectories will often be full ofpylint
suggestions. This change adds the relevant subdirectories to thepylintrc
file. Note that this does not necessarily enable us to runpylint
directly on those subdirectories with the equivalentmake
commands to the ones that already exist, because there is no way that I've found to makepylint
ignore the__init__.py
file of whatever module it's being run on, so running it onhail/python/hail
, for example, produces many errors.