summaryrefslogtreecommitdiff
path: root/CONTRIBUTORS.txt
Commit message (Collapse)AuthorAgeFilesLines
...
* Add a checker for misleading unicode (#5311)Kound2022-01-131-1/+2
| | | | Co-authored-by: Pierre Sassoulas <pierre.sassoulas@gmail.com> Co-authored-by: Daniël van Noord <13665637+DanielNoord@users.noreply.github.com>
* Improve non ascii checker (#5643)Kound2022-01-101-0/+3
| | | | | | | | | | * split ``non-ascii-name`` into 3 different msgs - non-ascii-identifier (replaces non-ascii-name) - non-ascii-file-name (a warning) - non-ascii-module-import (only considering the namespace the import is imported in) Co-authored-by: Daniël van Noord <13665637+DanielNoord@users.noreply.github.com> Co-authored-by: Pierre Sassoulas <pierre.sassoulas@gmail.com>
* Fix typos over the whole codebase (#5540)Kian Meng, Ang2021-12-171-2/+4
| | | Co-authored-by: Daniël van Noord <13665637+DanielNoord@users.noreply.github.com>
* Fix assigning-non-slot false positive with setattr (#5457)Jake Lishman2021-12-151-0/+2
| | | | | | | | | | | | * Fix assigning-non-slot false positive with setattr Previously, if a class was slotted and overrode `__setattr__`, `assigning-non-slot` would be issued when assigning to attributes. With `__setattr__` defined, we cannot infer if it is an error to assign to an attribute, so we suppress the error. Fix #3793 Co-authored-by: Pierre Sassoulas <pierre.sassoulas@gmail.com>
* typecheck: simplify variadic positional detection (#5417)Jérome Perrin2021-12-141-0/+2
| | | | | | | | The isinstance checks were not necessary here and caused several false positives where a function with variadic positional argument was called, like for example when the call is used as a function argument, in a if, while or with statement. Co-authored-by: Pierre Sassoulas <pierre.sassoulas@gmail.com>
* Docker: Python 3.10 and Alpine 3.15 (#5443)oittaa2021-12-031-0/+2
| | | | * Docker: Python 3.10 and Alpine 3.15
* Update outdated class name in "How to Write a Checker" test example (#5449)Philipp Albrecht2021-12-031-0/+2
| | | | | | | | | | | | * Update code example in "How to Write a Checker" `pylint.testutils.Message` got first renamed to `pylint.testutils.TestMessage` in 49776acdd and finally renamed to `pylint.testutils.MessageTest` in 24cbf8c33. * Add pylbrecht to contributors Co-authored-by: Philipp Albrecht <philipp.albrecht@momox.biz> Co-authored-by: Pierre Sassoulas <pierre.sassoulas@gmail.com>
* #5452: Fix false positive missing-doc-param from multi-line Google-st… (#5459)allanc652021-12-031-0/+3
| | | | | * #5452: Fix false positive missing-doc-param from multi-line Google-style docstrings. Co-authored-by: Daniël van Noord <13665637+DanielNoord@users.noreply.github.com>
* Clarify syntax in pylint.lint example (#5260)Felix von Drigalski2021-11-131-1/+3
| | | | | * Clarify syntax in pylint.lint example This makes the call non-trivial and demonstrates the syntax more fully.
* Pylint fix for invalid TOML config (#4720)Tanvi Moharir2021-11-131-0/+3
| | | | | | | | | | | | * Fix crashes during toml configuration parsing Add test for current 'pyproject.toml' issues. Add a 'bad-configuration-section' message for bad toml configuration We can detect bad top level option when reading the toml but we do not catch all the problem in toml because we don't know what is expected so we can't recommend. See #5259 Co-authored-by: Daniël van Noord <13665637+DanielNoord@users.noreply.github.com>
* Inspection for `with threading.Lock():` (#5245)Martin2021-11-031-0/+1
| | | | | | | | | | | | | | | | | | | Using `with threading.Lock():` directly has no effect. Correct usage is: ``` lock = threading.Lock() with lock: ... ``` This applies for: * threading.Lock * threading.RLock * threading.Condition * threading.Semaphore * threading.BoundedSemaphore Signed-off-by: Martin Basti <mbasti@redhat.com>
* Use single worker if importing `multiprocessing.synchronize` fails (#5238)Harshil2021-11-031-0/+2
| | | | | * add test import for multiprocessing.synchronize Co-authored-by: Jacob Walls <jacobtylerwalls@gmail.com>
* add changelog and contributor entryAntonio Quarta2021-10-311-0/+2
|
* Fix ``protected-access`` for attributes and methods of nested classes (#5232)Daniël van Noord2021-10-311-0/+2
| | | | | | | | | | | | | | | * Fix access to private function in inner class on protected-access bug * Add functional test for protected-access from inner class * Add Ikraduya to CONTRIBUTORS file * Add if statement to avoid potential bug * Fix ``protected-access`` for attributes and methods of nested classes This closes #3066 Co-authored-by: ikraduya <ikraduya@gmail.com> Co-authored-by: Pierre Sassoulas <pierre.sassoulas@gmail.com>
* Unify all `<y or n>` metavars in option's help (#5229)Tushar Sadhwani2021-10-291-0/+2
| | | | * Unify all `<y or n>` metavars
* Clean up some useless-super-delegation logic (#5193)Samuel Freilich2021-10-261-0/+2
| | | | | | | | | | | | | | * Clean up some useless-super-delegation logic In _definition_equivalent_to_call, avoid constructing a tuple and calling "all" on that. Instead, just check each condition in turn, terminating as soon as one fails. In _has_different_parameters_default_value, use a single sentinel value for a missing default instead of constructing a new one each call. Also construct the dict of comparators once instead of once each call. Restructure the logic to be less nested. Co-authored-by: Daniël van Noord <13665637+DanielNoord@users.noreply.github.com>
* Fix exception of pyreverse in handling property function within a class.Takahide Nojima2021-10-261-0/+2
|
* Add configuration option ``exclude-too-few-public-methods`` (#5191)Mike Fiedler2021-10-261-0/+2
| | | | | | | | | Allow excluding classes based on their ancestors from the ``too-few-public-methods`` checker. Closes #3370 Signed-off-by: Mike Fiedler <miketheman@gmail.com> Co-authored-by: Daniël van Noord <13665637+DanielNoord@users.noreply.github.com>
* Add ``mixin-class-rgx`` option (#5203)Daniël van Noord2021-10-251-0/+2
| | | | | Co-authored-by: Alpha <alpha@pokesplash.net> Co-authored-by: Pierre Sassoulas <pierre.sassoulas@gmail.com>
* Add extension checker that suggests any/all statements from for loops (#5196)Arianna Y2021-10-251-0/+2
| | | | | | | | | * Add extension checker that suggests any/all statements from for loops * Suggest all/not all: If there are negated conditions, we can suggest an all/not all statement to move the 'not' out of the comprehension and call it only once. Co-authored-by: Pierre Sassoulas <pierre.sassoulas@gmail.com>
* Fix useless-super-delegation false positive when default keyword argument is ↵Nick Pesce2021-10-191-0/+2
| | | | | | | a variable. (#5157) Compare variable default args and simplify the logic of the checkers. Co-authored-by: Pierre Sassoulas <pierre.sassoulas@gmail.com>
* Async context manager typo (#5164)Youngsoo Sung2021-10-171-0/+2
|
* Consider Enums when checking for duplicate dictionary keys (#5155)jaydesl2021-10-161-0/+2
| | | | | * detect duplicate-key for enum members Co-authored-by: Pierre Sassoulas <pierre.sassoulas@gmail.com>
* Rename `len-as-condition` to `use-implicit-booleaness-not-len` (#5132)Jaehoon Hwang2021-10-101-0/+2
| | | | | | | | | | Rename `len-as-condition` to be more general for new checker `use-implicit-booleaness-not-comparison` * Refactor `LenChecker` class -> `ImplicitBooleanessChecker`o * Rename test files/`len_checker.py`/`__init__.py` to reflect new name. * Add `len-as-condition` as `old_names` for `use-implicit-booleaness-not-len` Co-authored-by: Pierre Sassoulas <pierre.sassoulas@gmail.com>
* Add support for `ModuleNotFoundError` Python 3.6+ builtin-exception (#5106)Samuel FORESTIER2021-10-091-0/+2
|
* Fix a crash if ``str.format`` is referenced without being called (#5109)Tim Martin2021-10-061-0/+2
| | | | | Closes #5058 Co-authored-by: Pierre Sassoulas <pierre.sassoulas@gmail.com>
* Update toml dependency to >=0.9.2 (#5067)Jeroen Seegers2021-09-231-0/+3
| | | | * Update toml dependency from >=0.7.1 to >=0.9.2
* Update CONTRIBUTORS.txtDimitri Prybysh2021-09-141-1/+1
| | | | I feel confident enough to fix my own last name.
* Fix no-self-use and docparams extension for async functions/methods (#4986)SupImDos2021-09-111-0/+4
| | | | | | | | | | | | | | | | * Fixed 'no-self-check' for async functions. * fixed 'docparams' for async functions. * Added 'no-self-use' to the pylint disable comment in broken test. * Added small changelog entry to Pylint 2.11.0 milestone. * Added test: 'no-self-use' for async method. * Added test: revamped 'docparams' functional tests to test all missing documentation. Co-authored-by: Marc Mueller <30130371+cdce8p@users.noreply.github.com> Co-authored-by: Pierre Sassoulas <pierre.sassoulas@gmail.com>
* Upgrade ignored-parents help description (#4926)Melvin2021-08-281-0/+2
| | | * Upgrade ignored-parents help description
* Added myself to CONTRIBUTORS.txtPhilipp A2021-08-261-0/+2
|
* [duplicate-code] Parse functions and class methods recursively when ↵Maksym Humetskyi2021-08-171-0/+1
| | | | | gathering signature lines (#4858) * [duplicate-code] Parse functions and class methods recursively when gathering signature lines
* [duplicate-code] Ignore decorators lines by similarities checker when ignore ↵Maksym Humetskyi2021-08-141-0/+1
| | | | | | | | | | | | signatures flag enabled (#4840) * [duplicate-code] Ignore decorators lines by similarities checker when ignore signatures flag enabled * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci Co-authored-by: Maksym Humetskyi <mhumets@softserveinc.com> Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
* Update option_manager_mixin.py - issue 3839 (#4812)PaaEl2021-08-111-0/+2
| | | | | | | | | | | | | * Update option_manager_mixin.py I added user 'JoshMayberry' 's solution for issue 3839. https://github.com/PyCQA/pylint/issues/3839 I added a test to see if pylint handles environment variables in config file locations. Added it to test_config because it also deals with config files. To test I added an environment variable containing tmp_path and then unpacked it with os.path.expandvars. Tmp_path gets converted to an environment variable which is fed to the read_config_file function to be tested. Read_config_file succesfully reads the environment variable, if the variable is changed, the function raises an OSError.
* Fix false positives for invalid-all-format (#4711) (#4829)kasium2021-08-111-0/+2
| | | * Fix false positives for invalid-all-format (#4711)
* Fix IsADirectoryError in tests/lint/unittest_lint (#4781)Michal Vasilek2021-08-011-0/+2
| | | pylintd is a directory, so os.remove throws IsADirectoryError
* Use `XDG_CACHE_HOME` for `PYLINTHOME` (#4661)Eisuke Kawashima2021-07-281-0/+2
| | | | | Closes #3878 Co-authored-by: Pierre Sassoulas <pierre.sassoulas@gmail.com>
* Add ignored-parents option to design checker (#4758)Rebecca Turner2021-07-281-0/+2
| | | | | | | | | | | * Add ignored-parents option to design checker This allows users to specify classes to ignore while counting parent classes. Partially closes #3057 Co-authored-by: Pierre Sassoulas <pierre.sassoulas@gmail.com> Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
* Add unspecified-encoding checker #3826 (#4753)Daniël van Noord2021-07-281-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | * Add unspecified-encoding checker #3826 This adds an unspecified-encoding checker that adds a warning whenever open() is called without an explicit encoding argument. This closes #3826 * Update tests to conform to unspecified-encoding With addition of the unspecified-encoding checker calls of open() need an encoding argument. Where necessary this argument has been added, or the message has been disabled. This also includes small linting changes to a small number of tests. Their test-data has been updated to reflect new line numbers. * Update scripts to conform to unspecified-encoding With addition of the unspecified-encoding checker calls of open() need an encoding argument. Where necessary this argument has been added. * Update pylint to conform to unspecified-encoding With addition of the unspecified-encoding checker calls of open() need an encoding argument. Where necessary this argument has been added. Co-authored-by: Marc Mueller <30130371+cdce8p@users.noreply.github.com> Co-authored-by: Marc Mueller <30130371+cdce8p@users.noreply.github.com>
* Fix false positive undefined variable in decorator with list comprehension ↵Marcin Kurczewski2021-07-221-0/+2
| | | | | | (#4737) Closes #3791
* Added `time.clock` to deprecated functions/methods for python 3.3.Yilei "Dolee" Yang2021-07-201-0/+2
| | | | It was deprecated in 3.3 and removed in 3.8.
* document Jupyter integration (#4714)Marco Edward Gorelli2021-07-181-0/+3
|
* Changed exit logic.Mark Bell2021-07-171-0/+2
|
* Clarify consider-using-from-import documentationWill Shanks2021-07-101-0/+2
| | | | consider-using-from-import only handles submodules
* Catch MRO error when applying missing hints feature. (#4688)doranid2021-07-081-0/+2
| | | | | * Also catch MroError. Co-authored-by: Pierre Sassoulas <pierre.sassoulas@gmail.com>
* [duplicate-code] Fix ignored empty functions by similarities checker with ↵Maksym Humetskyi2021-07-051-0/+3
| | | | | ignore-signatures option enabled (#4669) Co-authored-by: Maksym Humetskyi <mhumets@softserveinc.com>
* VariableChecker now accounts for attribute lookups in type comments (#4604)Sergei Lebedev2021-06-271-0/+2
| | | | | | | | | | | * VariableChecker now accounts for attribute lookups in type comments Prior to this commit VariableChecker did not recurse into attribute lookups in type comments. This lead to false positive unused-import messages in e.g. import collections d = ... # type: collections.OrderedDict Fixes #4603
* Remove docs and code associated with the removed W0623 (redefine-in-handler) ↵Lorena B2021-06-201-0/+2
| | | | | checker (#4543) * remove docs and code associated with the removed W0623 checker
* Handle floats when parsing pyproject.toml (#4518) (#4520)Markus Siebenhaar2021-05-291-0/+2
| | | | | | | | | | * Handle floats when parsing pyproject.toml (#4518) * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci Co-authored-by: Markus Siebenhaar <markus.siebenhaar@masiha.de> Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
* Add `ignore-paths` to match against the full path. (#4516)Fabian Damken2021-05-271-0/+4
| | | | | | | | | | | | * Add ``ignore-paths`` configuration directive List of regex matching against the full path Close #2541 * Adjusted previous changes for current pylint version. Co-authored-by: Bernard Nauwelaerts <bernard@cybernet.be> Co-authored-by: Pierre Sassoulas <pierre.sassoulas@gmail.com>