summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
...
* Update coverage requirement from ~=5.5 to ~=6.1.2 (#5239)dependabot[bot]2021-11-111-1/+1
| | | | | | | | | | | | | | | | | | | | * Update coverage requirement from ~=5.5 to ~=6.1.2 Updates the requirements on [coverage](https://github.com/nedbat/coveragepy) to permit the latest version. - [Release notes](https://github.com/nedbat/coveragepy/releases) - [Changelog](https://github.com/nedbat/coveragepy/blob/master/CHANGES.rst) - [Commits](https://github.com/nedbat/coveragepy/compare/coverage-5.5...6.1.1) --- updated-dependencies: - dependency-name: coverage dependency-type: direct:production ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Pierre Sassoulas <pierre.sassoulas@gmail.com> Co-authored-by: Marc Mueller <30130371+cdce8p@users.noreply.github.com>
* Use a stable file in order to have a stable exit codePierre Sassoulas2021-11-102-1/+7
|
* Refactor 'check_configuration_file_reader' so we can assert other filesPierre Sassoulas2021-11-101-18/+30
| | | | Necessary prior to #4720
* Fix the typing of all PosixPath in tests/config/test_config.pyPierre Sassoulas2021-11-101-6/+6
|
* Simplify 'test_can_read_toml_env_variable' to work like other testsPierre Sassoulas2021-11-101-19/+6
| | | | | | | See initial intent in https://github.com/PyCQA/pylint/issues/3839 This permit to not use the OptionManagerMixin directly, which is problematic when it needs the function defined in Pylinter or other classes.
* Add checker `using-final-decorator-in-unsupported-version` (#5165)Mark Byrne2021-11-0912-13/+148
| | | | | | | | | | | | | | | | | | | | * Add checker `using-final-in-unsupported-version` This is one of the tasks in issue: #5134 Also: - Ensure the existing checkers for `typing.final` are used irrespective of Python version * Emit `using-final-in-unsupported-version` warning when Python version < 3.8 and none of the other `typing.final`-related warnings * Add `uninferable_final_decorators` Return any `typing.final` decorators for a given `Decorators` node. Used to determine if this decorator is used with a version of Python in which it is unsupported. Co-authored-by: Pierre Sassoulas <pierre.sassoulas@gmail.com>
* [pre-commit.ci] pre-commit autoupdatepre-commit-ci[bot]2021-11-081-1/+1
| | | | | updates: - [github.com/PyCQA/isort: 5.9.3 → 5.10.0](https://github.com/PyCQA/isort/compare/5.9.3...5.10.0)
* Bump actions/checkout from 2.3.5 to 2.4.0dependabot[bot]2021-11-083-13/+13
| | | | | | | | | | | | | | | Bumps [actions/checkout](https://github.com/actions/checkout) from 2.3.5 to 2.4.0. - [Release notes](https://github.com/actions/checkout/releases) - [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md) - [Commits](https://github.com/actions/checkout/compare/v2.3.5...v2.4.0) --- updated-dependencies: - dependency-name: actions/checkout dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com>
* Bump isort from 5.9.3 to 5.10.0dependabot[bot]2021-11-081-1/+1
| | | | | | | | | | | | | | | Bumps [isort](https://github.com/pycqa/isort) from 5.9.3 to 5.10.0. - [Release notes](https://github.com/pycqa/isort/releases) - [Changelog](https://github.com/PyCQA/isort/blob/main/CHANGELOG.md) - [Commits](https://github.com/pycqa/isort/compare/5.9.3...5.10.0) --- updated-dependencies: - dependency-name: isort dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com>
* Simplify toml handling by removing else after tryPierre Sassoulas2021-11-081-13/+12
|
* Rename test_can_read_env_variable so it has 'toml' in itPierre Sassoulas2021-11-081-1/+2
| | | | It permits to launch the toml test easily with pytest -k toml
* Remove duplicated comment in 'test_can_read_env_variable'Pierre Sassoulas2021-11-081-2/+0
|
* Move existing config tests to the tests/config directoryPierre Sassoulas2021-11-082-0/+0
|
* Create a function for toml parsingPierre Sassoulas2021-11-081-23/+28
| | | | Handle https://github.com/PyCQA/pylint/pull/4720\#discussion_r742853160
* Create and use a function for module stats initialization (#5271)Pierre Sassoulas2021-11-082-19/+15
| | | | | | | This permit to reduce the coupling between Pylinter and linterstats. Also add two missing litteral in typing for module stats and independant typing for ModuleStats attribute Refactor prior to #4720
* Make ``self-cls-assignment`` check tuple assignment (#5268)Daniël van Noord2021-11-075-20/+23
| | | Co-authored-by: Marc Mueller <30130371+cdce8p@users.noreply.github.com>
* Fix crash on checking private members on ``__class__``Daniël van Noord2021-11-054-6/+27
| | | | Closes #5261
* Remove ``render: python`` from bug descriptionDaniël van Noord2021-11-041-1/+0
|
* Update ``coveralls`` requirement to ~=3.3 (#5257)Daniël van Noord2021-11-041-1/+1
|
* Remove unwanted non-breaking whitespaces in comment (#5256)Pierre Sassoulas2021-11-043-13/+13
|
* Add script to generate new message category id (#5248)Daniël van Noord2021-11-043-0/+53
|
* Inspection for `with threading.Lock():` (#5245)Martin2021-11-036-0/+134
| | | | | | | | | | | | | | | | | | | 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-034-0/+13
| | | | | * add test import for multiprocessing.synchronize Co-authored-by: Jacob Walls <jacobtylerwalls@gmail.com>
* [pre-commit.ci] pre-commit autoupdate (#5243)pre-commit-ci[bot]2021-11-011-1/+1
| | | | | | updates: - [github.com/psf/black: 21.9b0 → 21.10b0](https://github.com/psf/black/compare/21.9b0...21.10b0) Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
* Bump black from 21.9b0 to 21.10b0 (#5240)dependabot[bot]2021-11-011-1/+1
| | | | | | | | | | | | | | | | Bumps [black](https://github.com/psf/black) from 21.9b0 to 21.10b0. - [Release notes](https://github.com/psf/black/releases) - [Changelog](https://github.com/psf/black/blob/main/CHANGES.md) - [Commits](https://github.com/psf/black/commits) --- updated-dependencies: - dependency-name: black dependency-type: direct:production ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* Merge pull request #5233 from sgheppy/nographviz_fixAndreas Finkler2021-10-314-1/+7
|\ | | | | Nographviz fix
| * add changelog and contributor entryAntonio Quarta2021-10-313-0/+6
| |
| * fix accepted output format without graphvizAntonio Quarta2021-10-311-1/+1
|/
* Remove accidental print statementDaniël van Noord2021-10-311-1/+0
|
* Fix ``protected-access`` for attributes and methods of nested classes (#5232)Daniël van Noord2021-10-317-16/+123
| | | | | | | | | | | | | | | * 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>
* Allow no type in Numpy param docstrings for ``mising-param-doc`` (#5231)Daniël van Noord2021-10-314-6/+53
| | | | | This closes #5222 See https://numpydoc.readthedocs.io/en/latest/format.html#parameters for reference
* Unify all `<y or n>` metavars in option's help (#5229)Tushar Sadhwani2021-10-2913-22/+24
| | | | * Unify all `<y or n>` metavars
* Fix incorrect ``consider-using-ternary`` when condition is inferable as ↵Arianna Y2021-10-295-2/+17
| | | | | | | False (#5227) * Fix incorrect ``consider-using-ternary`` when condition is inferrable as False * Properly infer the condition in old ternary statements and suggest ``simplify-boolean-expression`` over ``consider-using-ternary`` if it is False
* Fix tox dependencies with pypy (#5217)nozzy123nozzy2021-10-271-1/+3
|
* Migration to pytest from unittest for BaseChecker testsPierre Sassoulas2021-10-271-37/+35
|
* Create a new file for BaseChecker unit testsPierre Sassoulas2021-10-272-61/+69
|
* Fix deprecation 'check function will only accept sequence of string' (#5218)Pierre Sassoulas2021-10-272-14/+24
| | | | | * Fix deprecation 'check function will only accept sequence of string' * Add a test for Pylinter.check() deprecation
* Fix the deprecation of set_output in our code (#5209)Pierre Sassoulas2021-10-274-20/+31
| | | | | | | | | | * Fix the deprecation of set_output in our code * Add a test for the deprecation warning itself * Refactor the set_output from the MultiReporter We can remove set_output declaration because the function was already failing with a NotImplementedError before, so removing it make it fail another way
* Clean up some useless-super-delegation logic (#5193)Samuel Freilich2021-10-262-61/+47
| | | | | | | | | | | | | | * 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>
* Merge pull request #5198 from nozzy123nozzy/bugfix_of_propertyAndreas Finkler2021-10-2618-1/+79
|\ | | | | Fix exception of pyreverse in handling property function within a class.
| * Update tests/data/property_pattern.pynozzy123nozzy2021-10-261-5/+0
| | | | | | | | | | | | Remove the "main" function because this is meaningless for testing. Co-authored-by: Andreas Finkler <3929834+DudeNr33@users.noreply.github.com>
| * Modify sources along with results of pr review.Takahide Nojima2021-10-2615-41/+64
| | | | | | | | | | | | | | - move "tests/prop_data/*" to "tests/data/." - make tests/data/property_pattern.py correctly executable. - remove class members not related to "property" from test/data/property_pattern.py. - modify tests of pyreverse to pass all tests.
| * Fix exception of pyreverse in handling property function within a class.Takahide Nojima2021-10-266-0/+60
|/
* Add control flow check for ``undefined-variable`` in ``if ... else``Daniël van Noord2021-10-265-1/+85
| | | | Closes #3688
* Add configuration option ``exclude-too-few-public-methods`` (#5191)Mike Fiedler2021-10-2610-1/+75
| | | | | | | | | 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>
* Move the find_pylintrc function in find_default_config_files.pyPierre Sassoulas2021-10-262-11/+14
|
* Create a OLD_DEFAULT_PYLINT_HOME constantsPierre Sassoulas2021-10-264-7/+12
|
* Create a DEFAULT_PYLINT_HOME constant and upgrade help textPierre Sassoulas2021-10-263-7/+8
|
* Put the environnement help directly where it makes sensePierre Sassoulas2021-10-262-13/+15
|
* Add tests for assignment expressions in function defaults (#5188)Daniël van Noord2021-10-2510-3/+91
| | | | | | | | | * Add tests for assignment expressions in function defaults Ref #3688 * Upgrade astroid to 2.8.4 Co-authored-by: Pierre Sassoulas <pierre.sassoulas@gmail.com>