Commit message (Collapse) | Author | Age | Files | Lines | ||
---|---|---|---|---|---|---|
... | ||||||
* | Remove problematic annotations | Claudiu Popa | 2018-09-19 | 1 | -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 Popa | 2018-09-19 | 1 | -1/+0 | |
| | | | | This reverts commit 9d224833f1052b3792df2765b52388e16cff4791. | |||||
* | Check if an updated typing version fixes readthedocs | Claudiu Popa | 2018-09-19 | 1 | -0/+1 | |
| | ||||||
* | New tagline! Many thanks to whomever said it, it came my way through DRMacIver | Claudiu Popa | 2018-09-19 | 1 | -0/+6 | |
| | ||||||
* | ``too-few-public-methods`` is not reported for ``typing.NamedTuple`` | Claudiu Popa | 2018-09-18 | 3 | -4/+26 | |
| | | | | Close #2459 | |||||
* | Move is_dataclass and is_enum_class back to where they are used | Claudiu Popa | 2018-09-18 | 2 | -59/+56 | |
| | ||||||
* | Correct fallback when the lines out of the parse trees are in ↵ | Claudiu Popa | 2018-09-18 | 1 | -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 black | Claudiu Popa | 2018-09-18 | 1 | -0/+1 | |
| | ||||||
* | Partially revert commit fede47a625f40169bf839b7e8e2df643b3471635 | Claudiu Popa | 2018-09-18 | 3 | -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 case | Claudiu Popa | 2018-09-17 | 1 | -1/+1 | |
| | ||||||
* | Fix mypy errors caused by black autoformatting the type comments | Claudiu Popa | 2018-09-17 | 2 | -4/+4 | |
| | ||||||
* | Add formatting documentation | Claudiu Popa | 2018-09-17 | 1 | -0/+10 | |
| | ||||||
* | Add the precommit configuration | Claudiu Popa | 2018-09-17 | 1 | -0/+12 | |
| | ||||||
* | Fix the tests broken by the disabling of formatting checks over pylint codebase | Claudiu Popa | 2018-09-17 | 1 | -2/+2 | |
| | ||||||
* | Add tox and travis black steps | Claudiu Popa | 2018-09-17 | 2 | -3/+12 | |
| | ||||||
* | Add black badge | Claudiu Popa | 2018-09-17 | 1 | -0/+3 | |
| | ||||||
* | Reformat the code with black | Claudiu Popa | 2018-09-17 | 87 | -8342/+12199 | |
| | ||||||
* | ```too-few-public-methods`` is not reported for dataclasses created with ↵ | Claudiu Popa | 2018-09-15 | 3 | -1/+14 | |
| | | | | | | options. Close #2488 | |||||
* | Move is_raising to where it is used and correct its annotation | Claudiu Popa | 2018-09-15 | 2 | -10/+13 | |
| | ||||||
* | Grammar fixes to docs (#2492) | Drew | 2018-09-15 | 5 | -6/+8 | |
| | | | Fix grammar error in pylintrc files and CONTRIBUTORS.txt. | |||||
* | Remove wrong modules from 'bad-python3-import' (#2454) | Aivar Annamaa | 2018-09-09 | 4 | -2/+26 | |
| | | | | Close #2453 | |||||
* | The ``json`` reporter prints an empty list when no messages are emitted | Claudiu Popa | 2018-09-06 | 2 | -2/+5 | |
| | | | | Close #2446 | |||||
* | Remove useless minversion for Python 2.7 | Claudiu Popa | 2018-09-05 | 1 | -12/+6 | |
| | ||||||
* | Add a new check, ``duplicate-string-formatting-argument`` | Claudiu Popa | 2018-09-05 | 5 | -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 around | Claudiu Popa | 2018-09-05 | 1 | -6/+3 | |
| | ||||||
* | Drop the .keys() call, can unpack dictionaries without it | Claudiu Popa | 2018-09-05 | 4 | -8/+7 | |
| | ||||||
* | Switch to using CallSite to get the arguments and clear the formatting a bit | Claudiu Popa | 2018-09-05 | 1 | -28/+15 | |
| | ||||||
* | Remove vacuous version check | Claudiu Popa | 2018-09-05 | 1 | -2/+1 | |
| | ||||||
* | Switch to a hasattr() call for getting the elts of an inferred value | Claudiu Popa | 2018-09-05 | 1 | -1/+1 | |
| | ||||||
* | Make sure that args_elts is defined by defaulting to an empty tuple | Claudiu Popa | 2018-09-05 | 1 | -0/+1 | |
| | ||||||
* | isinstance accept tuples of tuples so no need to join them | Claudiu Popa | 2018-09-05 | 1 | -2/+2 | |
| | ||||||
* | Remove extraneous comment and fix the style | Claudiu Popa | 2018-09-05 | 1 | -6/+2 | |
| | ||||||
* | ``assignment-from-no-return`` is not emitted for coroutines. | Claudiu Popa | 2018-09-05 | 5 | -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 Popa | 2018-09-05 | 4 | -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 Gribov | 2018-09-04 | 7 | -7/+101 | |
| | ||||||
* | Disable pylint error locally | Claudiu Popa | 2018-09-04 | 1 | -1/+1 | |
| | ||||||
* | Merge documentation with contributing docs | Claudiu Popa | 2018-09-04 | 3 | -10/+8 | |
| | ||||||
* | Switch to Unix line endings | Claudiu Popa | 2018-09-04 | 1 | -29/+29 | |
| | ||||||
* | Handle ``AstroidSyntaxError`` when trying to import a module. | Claudiu Popa | 2018-09-04 | 4 | -0/+12 | |
| | | | | Close #2313 | |||||
* | Fix incorrect package check for wildcard imports | Hornwitser | 2018-09-04 | 2 | -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 tests | Hornwitser | 2018-09-04 | 5 | -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..." | Hornwitser | 2018-09-04 | 1 | -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 broken | Hornwitser | 2018-09-04 | 2 | -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 graph | Hornwitser | 2018-09-04 | 3 | -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 doc | Roberto Leinardi | 2018-09-04 | 1 | -0/+1 | |
| | ||||||
* | Updated pylint-pycharm plugin documentation | Roberto Leinardi | 2018-09-03 | 1 | -8/+10 | |
| | ||||||
* | Allow ``__module__`` to be redefined at a class level. Close #2451 | Claudiu Popa | 2018-08-31 | 3 | -2/+32 | |
| | ||||||
* | Emit unused-import instead of unused-variable when dealing with unused imports | Sushobhit | 2018-08-31 | 6 | -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 Popa | 2018-08-24 | 4 | -2/+71 | |
| | | | | Close #996 | |||||
* | Remove last occurrences of locally-enabled | Claudiu Popa | 2018-08-24 | 2 | -5/+0 | |
| |