summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
...
* improve NotCheckerLaura M?dioni2015-10-271-5/+8
| | | | | - move reverse_op declaration out of visit_unaryop - build message out of operand, instead of using replace() on node.as_string()
* improve unneeded-not messagesLaura M?dioni2015-10-272-14/+14
| | | | | - warning => convention - "One not too many" => "consider changing.."
* Check for nots too many in comparison expressionsLaura M?dioni2015-10-265-2/+86
|
* Ignore missing docstrings for decorated attribute setters and deletersMichael Kefeder2015-11-192-0/+31
| | | | Closes issue #651.
* Improve the install section of the README, by specifying pip and how to use ↵Claudiu Popa2015-11-221-4/+9
| | | | the master development head.
* Cleanup pylint issuesClaudiu Popa2015-11-2110-39/+22
| | | | | | | | | | | This changeset also brings a couple of changes: * rrheaders and rcheaders are dropped from html_writer.Table's constructor. They weren't used at all and it was dead code. This simplified some if statements. * _is_attribute_property is used to look for a property assignment instead on relying on a different implementation.
* Added a new refactoring warning, 'simplifiable-if-statement'Claudiu Popa2015-11-214-1/+211
| | | | | | | | | | | | | | This is used when an if statement could be reduced to a boolean evaluation of its test, as seen in this example: if some_cond: return True else: return False could be reduced to `return bool(some_cond)` Closes issue #698.
* Use the correct module.Claudiu Popa2015-11-201-2/+2
|
* Simplify the unflatten algorithm in order to not use NodeNG anymore.Claudiu Popa2015-11-201-5/+7
|
* Import has_known_bases and safe_infer back into pylint from astroid, until ↵Claudiu Popa2015-11-2012-74/+106
| | | | | | | | | the latter stabilizes its API. Currently astroid goes into a total revamp, having a couple of development branches with partially incompatible APIs, which means that pylint can't rely on the exact location of has_known_bases and safe_infer until astroid reaches a new major release. With this in mind, these two functions are backported in pylint again.
* Merged in anentropic/pylint/remove-E0201-references (pull request #303)Claudiu Popa2015-11-184-4/+4
|\ | | | | | | error E0201 is obsolete - now E1101
| * 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.
View Lorry Controller Status