summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
...
| * error E0201 is obsolete - now E1101remove-E0201-referencesAnentropic2015-11-184-4/+4
| |
* | check the number of boolean expressions in if statement is reasonnableLaura M?dioni2015-10-295-3/+73
| | | | | | | | | | | | | | --max-bool-expr option allows to configure it (by default, up to 5 are tolerated) closes issue #677
* | Use safe inference in unpacking-non-sequence checkerDmitry Pribysh2015-11-095-27/+37
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Unfortunately, this fix means that we won't be able to emit an error for cases like this one: ``` def foo(): if True: return [1, 2] return [3, 4, 5] a, b = foo() ``` Well, not unless we get flow-sensitive inference. But we still need this fix to reduce the number of false-positive errors. Fixes issue #695.
* | Refactor non-iterator-returned checker to use safe inferenceDmitry Pribysh2015-11-091-8/+24
| | | | | | | | This way it won't infer all possibles values of a node, only two at most.
* | fix get_type()Laura M?dioni2015-11-061-1/+1
| | | | | | | | | | It returned erroneous number of comment lines. tok_type is set just once, here one must look at the current token
* | Remove pudb from test dependenciesDmitry Pribysh2015-11-071-1/+1
| |
* | Revert accidentally updated tox.ini fileDmitry Pribysh2015-11-071-1/+1
| |
* | Reduce number of false positives emitted by non-iterator-returned checkerDmitry Pribysh2015-11-074-19/+27
| | | | | | | | | | | | | | | | By disabling checker for cases when there're multiple possible values infered for node. This way we lose some of the inference power, but throw a lot less false positives. Fixes issue #695.
* | MergeClaudiu Popa2015-11-052-2/+17
|\ \
| * | display properties as attributes in pyreverse class diagramspaceone2015-10-122-2/+16
| | |
* | | Fix the test to work on both Python versions.Claudiu Popa2015-11-041-1/+1
| | |
* | | Add test for deques to `unsubscriptable-object` functional testsDmitry Pribysh2015-11-041-0/+6
| |/ |/|
* | Add ChangeLog entry for too-many-nested-blocks and fix the errors in pylint.Claudiu Popa2015-11-046-106/+113
| |
* | Merged in lmedioni/pylint (pull request #298)Claudiu Popa2015-11-044-3/+218
|\ \ | | | | | | | | | check if the number of nested block in a function or a method isn't too high
| * | some more comments for too-many-nested-blocks rule implementationLaura M?dioni2015-11-031-0/+7
| | | | | | | | | | | | related to the issue #668
| * | add an extra sentence saying why blocks nesting is a bad patternLaura M?dioni2015-11-031-1/+2
| | | | | | | | | | | | related to the issue #668
| * | also count if nodes from ifexp and list comprehensionLaura M?dioni2015-11-021-0/+7
| | | | | | | | | | | | related to the issue #668
| * | check if the number of nested block in a function or a method isn't too highLaura M?dioni2015-10-304-3/+203
| | | | | | | | | | | | related to issue #668
* | | Fix pylint warning by rewriting part of BasicCheckerDmitry Pribysh2015-11-031-4/+2
| | |
* | | Rename 'unsubscriptable-value' message to 'unsubscriptable-object'Dmitry Pribysh2015-11-034-33/+33
| | |
* | | Add checker for unsubscriptable values used in subscript expression.Dmitry Pribysh2015-11-036-2/+144
|/ / | | | | | | Fixes issue #561.
* | Fix an occurrence of no-classmethod-decorator.Claudiu Popa2015-10-301-1/+1
| |
* | Remove unused import.Claudiu Popa2015-10-301-1/+1
| |
* | Separe the conditions to improve the readability of the code.Claudiu Popa2015-10-301-0/+4
| |
* | Prefer any instead of for.Claudiu Popa2015-10-301-4/+3
| |
* | Fix the tests to work on both Python versions.Claudiu Popa2015-10-302-2/+2
| |
* | Add ChangeLog entry for issue #675.Claudiu Popa2015-10-301-0/+6
| |
* | Merged in lmedioni/pylint (pull request #295)Claudiu Popa2015-10-3013-8/+133
|\ \ | | | | | | | | | check for class methods declared without a decorator
| * | no-static/class-method: enhance the tests and fix the code accordinglyLaura M?dioni2015-10-295-7/+24
| | |
| * | improve style and fix typos regarding no_class/staticmethod_decoratorLaura M?dioni2015-10-293-4/+5
| | |
| * | check for static methods declared without a decoratorLaura M?dioni2015-10-297-9/+46
| | | | | | | | | | | | closes issue #675
| * | check for class methods declared without a decoratorLaura M?dioni2015-10-299-6/+76
| | | | | | | | | | | | related to the issue #675
* | | Merged in dmand/pylint/enhance-unpacking-checker (pull request #297)Claudiu Popa2015-10-305-155/+187
|\ \ \ | | | | | | | | | | | | Enhance tuple unpacking checker
| * | | Add a few functional tests for enhanced unpacking checkerenhance-unpacking-checkerDmitry Pribysh2015-10-292-13/+36
| | | |
| * | | Make tuple unpacking checker emit less false negativesDmitry Pribysh2015-10-293-142/+151
|/ / / | | | | | | | | | By reusing functions from iterable checker.
* | | Use the new master bookmark when installing astroid.Claudiu Popa2015-10-281-3/+3
| | |
* | | Fix pylint warnings.Claudiu Popa2015-10-271-4/+4
| | |
* | | Merge heads.Claudiu Popa2015-10-275-46/+168
|\ \ \
| * | | Add a new convention message, 'consider-using-enumerate'Claudiu Popa2015-10-274-0/+117
| | | | | | | | | | | | | | | | | | | | | | | | The message is emitted when code that uses `range` and `len` for iterating is encountered, which can be easily simplified by using `enumerate` instead. This makes the code a bit faster and cleaner. Closes issue #684.
| * | | Move unidiomatic-typecheck into the comparison checker from base.py, since ↵Claudiu Popa2015-10-262-46/+51
| |/ / | | | | | | | | | it makes more sense for it to be there.
* | | Merged in dmand/pylint/fix-685 (pull request #294)Claudiu Popa2015-10-277-107/+220
|\ \ \ | |/ / |/| | | | | Make iterable checker skip more abstract classes
| * | Make iterable checker skip classes that are inferred to be abstractfix-685Dmitry Pribysh2015-10-277-25/+121
| | |
| * | Skip not-an-iterable and similar checks for Base/Abstract classesDmitry Pribysh2015-10-272-84/+101
|/ / | | | | | | | | | | And refactor iterable checker. Fixes issue #685.
* | Simplify visit_compare, by moving the verifications for misplaced constants ↵Claudiu Popa2015-10-261-4/+4
| | | | | | | | into their own if statement.
* | Use the representation of the context when building the message for ↵Claudiu Popa2015-10-263-2/+4
| | | | | | | | misplaced-comparison-constant.
* | Add a Changelog entry for misplaced-comparison-constant.Claudiu Popa2015-10-261-0/+7
| |
* | Simplify _check_misplaced_constant.Claudiu Popa2015-10-261-8/+7
| |
* | Don't emit misplaced-comparison-constant for constant tests, such as true == ↵Claudiu Popa2015-10-263-15/+6
| | | | | | | | true, which should be handled by other checks.
* | Use a lower message id for misplaced-comparison-constant.Claudiu Popa2015-10-261-1/+1
| |
* | Make check_singleton_comparison a private function.Claudiu Popa2015-10-261-3/+3
| |