summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
...
* Remove problematic annotationsClaudiu Popa2018-09-191-10/+4
| | | | | | | | | | Due to unforeseen circumstances, readthedocs cannot render our documentation because we use a certain typing construction that errors on the Python 3.5 version that ReadTheDocs uses. We need to import pylint in order to construct the dynamic list of features, so the only choice right now is to remove these annotations from the codebase, although it would be great if ReadTheDocs would use a more recent version of Python: https://github.com/python/typing/issues/266
* Revert "Check if an updated typing version fixes readthedocs"Claudiu Popa2018-09-191-1/+0
| | | | This reverts commit 9d224833f1052b3792df2765b52388e16cff4791.
* Check if an updated typing version fixes readthedocsClaudiu Popa2018-09-191-0/+1
|
* New tagline! Many thanks to whomever said it, it came my way through DRMacIverClaudiu Popa2018-09-191-0/+6
|
* ``too-few-public-methods`` is not reported for ``typing.NamedTuple``Claudiu Popa2018-09-183-4/+26
| | | | Close #2459
* Move is_dataclass and is_enum_class back to where they are usedClaudiu Popa2018-09-182-59/+56
|
* Correct fallback when the lines out of the parse trees are in ↵Claudiu Popa2018-09-181-1/+16
| | | | | | | | | raw_module_msgs_state `_raw_module_msgs_state` holds a mapping of messages to lines and their status, enabled or disabled. As such, the correct solution is to check if any of those lines matches the current line that's out of the AST, as well as to check if it's disabled or enabled, since it can vary depending on the pragma that was used.
* Exclude the relevant directories from blackClaudiu Popa2018-09-181-0/+1
|
* Partially revert commit fede47a625f40169bf839b7e8e2df643b3471635Claudiu Popa2018-09-183-29/+13
| | | | | | | | | This used to assume that the initial part of a pragma line might have been a comment, but in practice in can be a bunch of control pragmas for different linters, which in turn results in pylint's pragma control no longer applying when it follows those pragma controls. Close #2297
* Rely on a different file than bad_continuation for the parallel test caseClaudiu Popa2018-09-171-1/+1
|
* Fix mypy errors caused by black autoformatting the type commentsClaudiu Popa2018-09-172-4/+4
|
* Add formatting documentationClaudiu Popa2018-09-171-0/+10
|
* Add the precommit configurationClaudiu Popa2018-09-171-0/+12
|
* Fix the tests broken by the disabling of formatting checks over pylint codebaseClaudiu Popa2018-09-171-2/+2
|
* Add tox and travis black stepsClaudiu Popa2018-09-172-3/+12
|
* Add black badgeClaudiu Popa2018-09-171-0/+3
|
* Reformat the code with blackClaudiu Popa2018-09-1787-8342/+12199
|
* ```too-few-public-methods`` is not reported for dataclasses created with ↵Claudiu Popa2018-09-153-1/+14
| | | | | | options. Close #2488
* Move is_raising to where it is used and correct its annotationClaudiu Popa2018-09-152-10/+13
|
* Grammar fixes to docs (#2492)Drew2018-09-155-6/+8
| | | Fix grammar error in pylintrc files and CONTRIBUTORS.txt.
* Remove wrong modules from 'bad-python3-import' (#2454)Aivar Annamaa2018-09-094-2/+26
| | | | Close #2453
* The ``json`` reporter prints an empty list when no messages are emittedClaudiu Popa2018-09-062-2/+5
| | | | Close #2446
* Remove useless minversion for Python 2.7Claudiu Popa2018-09-051-12/+6
|
* Add a new check, ``duplicate-string-formatting-argument``Claudiu Popa2018-09-055-1/+41
| | | | | | | This new check is emitted whenever a duplicate string formatting argument is found. Close #497
* Squash some checks and move some of the aroundClaudiu Popa2018-09-051-6/+3
|
* Drop the .keys() call, can unpack dictionaries without itClaudiu Popa2018-09-054-8/+7
|
* Switch to using CallSite to get the arguments and clear the formatting a bitClaudiu Popa2018-09-051-28/+15
|
* Remove vacuous version checkClaudiu Popa2018-09-051-2/+1
|
* Switch to a hasattr() call for getting the elts of an inferred valueClaudiu Popa2018-09-051-1/+1
|
* Make sure that args_elts is defined by defaulting to an empty tupleClaudiu Popa2018-09-051-0/+1
|
* isinstance accept tuples of tuples so no need to join themClaudiu Popa2018-09-051-2/+2
|
* Remove extraneous comment and fix the styleClaudiu Popa2018-09-051-6/+2
|
* ``assignment-from-no-return`` is not emitted for coroutines.Claudiu Popa2018-09-055-1/+35
| | | | | | | | The reason for that is that the function calls actually are returning something, a future object that needs to be consumed through the event loop, so emitting this check for coroutines is wrong. Close #1715
* ``consider-using-ternary`` and ``simplified-boolean-expression`` no longer ↵Claudiu Popa2018-09-054-30/+12
| | | | | | | | emit for sequence based checks This was more of an heuristic than actual an useful check, so let's get rid of it. Close #2473
* Added checker for format string type mismatches.Yury Gribov2018-09-047-7/+101
|
* Disable pylint error locallyClaudiu Popa2018-09-041-1/+1
|
* Merge documentation with contributing docsClaudiu Popa2018-09-043-10/+8
|
* Switch to Unix line endingsClaudiu Popa2018-09-041-29/+29
|
* Handle ``AstroidSyntaxError`` when trying to import a module.Claudiu Popa2018-09-044-0/+12
| | | | Close #2313
* Fix incorrect package check for wildcard importsHornwitser2018-09-042-2/+5
| | | | | | | | The root node name is the name of the package for __init__.py modules, remove incorect check for the name being equal to "__init__" leaving only the astdroid.Module.package attribute check. Fixes #2430
* Clean up imports in import checker unit testsHornwitser2018-09-045-21/+9
| | | | | | | | | | | Use astroid.MANAGER.ast_from_module_name instead of trying to manually build the module object. Rename awesome_module.py to wildcard.py to better reflect what's being tested, add empty.py and and do wildcard import tests importing it. Fixes incorrect module name passed in the test_wildcard_import_init unit test, as well as removing a self import, both of which were introduced at the creation of the test in 9c551b0.
* Revert "Don't emit import-self and cyclic-import..."Hornwitser2018-09-041-9/+0
| | | | | | | | | Reverts commit bcf801f "Don't emit import-self and cyclic-import for relative imports of modules with the same name as the package itself." The value from astroid.modutils.get_module_part no longer returns just the package from a module import, as indicated by the unit tests passing without the filtering code.
* Fix internal/external distinction being brokenHornwitser2018-09-042-16/+26
| | | | | | | | | | | | | | Partially introduced by a3d56dd, the logic for distinguishing internal and external dependences is broken due to a failure to account for multiple modules and packages being tested at once by pylint. Add a mapping of module to package it belongs to in the import checker and use this mapping to determine if an import is internal. An import is considered internal if the module imported is a submodule of the package it was imported from. Also consolidate filter logic to one function and replace the manual caching with astroid.decorators.cached.
* Fix wildcard imports missing in import graphHornwitser2018-09-043-1/+7
| | | | | This looks like an oversight in the graph adding code. '*' was explicitly checked for, but nothing was done on '*' imports.
* Added missin Info type to docRoberto Leinardi2018-09-041-0/+1
|
* Updated pylint-pycharm plugin documentationRoberto Leinardi2018-09-031-8/+10
|
* Allow ``__module__`` to be redefined at a class level. Close #2451Claudiu Popa2018-08-313-2/+32
|
* Emit unused-import instead of unused-variable when dealing with unused importsSushobhit2018-08-316-16/+46
| | | | | | | | When pylint encounters unused imports in scopes (e.g. functions), it used to emit `unused-variable`. This is somewhat confusing, as those names are not necessarily variables, so instead let's emit the more obvious `unused-import`. Close #2421
* Handle asyncio.coroutine when looking for ``not-an-iterable`` check.Claudiu Popa2018-08-244-2/+71
| | | | Close #996
* Remove last occurrences of locally-enabledClaudiu Popa2018-08-242-5/+0
|