summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Simplification in OptionManagerMixin following reviewadd-functional-tests-for-configurationPierre Sassoulas2021-11-121-3/+2
| | | | See https://github.com/PyCQA/pylint/pull/5287\#discussion_r748115210
* Use the existing function for relative path instead of a custom onePierre Sassoulas2021-11-121-9/+5
|
* Proper import for python import of CaptureFixturePierre Sassoulas2021-11-124-5/+4
| | | | See https://github.com/PyCQA/pylint/pull/5287\#discussion_r748122852
* Remove option that are not required for the test itselfPierre Sassoulas2021-11-121-5/+0
| | | | See https://github.com/PyCQA/pylint/pull/5287\#discussion_r748118048
* Add proper typing in configuration test following reviewPierre Sassoulas2021-11-121-1/+1
| | | | See https://github.com/PyCQA/pylint/pull/5287\#discussion_r748064197
* Apply suggestions from code reviewPierre Sassoulas2021-11-122-3/+3
| | | Co-authored-by: Daniël van Noord <13665637+DanielNoord@users.noreply.github.com>
* Add a regression test for #4746Pierre Sassoulas2021-11-125-5/+20
| | | | This permits to introduce an example of configuration file with an error.
* Rename variable that was still named for toml onlyPierre Sassoulas2021-11-121-2/+2
|
* Simplification of for loop on dict by using items()Pierre Sassoulas2021-11-121-2/+1
|
* Add a docstring to get_expected_or_default following reviewPierre Sassoulas2021-11-121-0/+2
| | | | See https://github.com/PyCQA/pylint/pull/5287\#discussion_r746971021
* Better explanation on why we use logging following reviewPierre Sassoulas2021-11-122-0/+8
| | | | See https://github.com/PyCQA/pylint/pull/5287#discussion_r746970574
* Simplification of for loop for functional conf test following reviewPierre Sassoulas2021-11-121-12/+11
| | | | See https://github.com/PyCQA/pylint/pull/5287#discussion_r747301970
* Create a constant for accepted configuration extensionPierre Sassoulas2021-11-121-1/+2
|
* More precise typing for Configuration type when Any is usedPierre Sassoulas2021-11-122-5/+13
|
* Factorize duplicated code from test_config using the new testutilPierre Sassoulas2021-11-123-33/+22
|
* Move test_can_read_toml_rich_types to functional frameworkPierre Sassoulas2021-11-123-20/+18
|
* Update pylint/testutils/configuration_test.pyPierre Sassoulas2021-11-121-2/+3
| | | | Co-authored-by: Daniël van Noord <13665637+DanielNoord@users.noreply.github.com>
* Apply suggestions from code reviewPierre Sassoulas2021-11-122-8/+8
| | | Co-authored-by: Daniël van Noord <13665637+DanielNoord@users.noreply.github.com>
* Easier understanding of what happen for the expected resultPierre Sassoulas2021-11-121-1/+12
|
* Simplify a list creation in OptionParser()Pierre Sassoulas2021-11-121-2/+1
|
* Better handling of removal in functional conf testsPierre Sassoulas2021-11-124-4/+37
|
* Create a framework of functional tests for configuration filesPierre Sassoulas2021-11-129-47/+202
| | | | Also migrate three standard unittest to the new framework for testing.
* Add tests for crash on assignment expressions in if statemenpyttsDaniël van Noord2021-11-123-0/+25
|
* Add tests for crash on inference of ``__len__``Daniël van Noord2021-11-121-0/+12
|
* Bump ``astroid`` version to ``2.8.5``Daniël van Noord2021-11-122-2/+2
|
* Add documentation on adding ``pylint`` to ``pre-commit`` (#5292)Daniël van Noord2021-11-123-0/+48
|
* 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>