search

opensearch-project / opensearch-py

Python Client for OpenSearch
https://opensearch.org/docs/latest/clients/python/
Apache License 2.0
359 stars 178 forks source link

[PROPOSAL] Replace lint_per_folder function in noxfile.py with .pylint files #645

Closed macohen closed 10 months ago

macohen commented 10 months ago

It may be more standard to use a .pylint file to specify pylint overrides. Advantages could be fewer runs of pylint, more standard. Disadvantages could be that all of those exceptions are peppered throughout the code.

Unless it's trivial this doesn't need to be part of this PR, but maybe there's a more generic solution where a .pylint file is placed into each folder with the configuration for that folder? This code could iterate over the folders and run it with the contents of that config file.

_Originally posted by @dblock in https://github.com/opensearch-project/opensearch-py/pull/643#discussion_r1446203242_

macohen commented 10 months ago

taking a look at moving my fun and maybe too clever code to config...

macohen commented 10 months ago

pylintrc files are in use.