Commit message (Collapse) | Author | Age | Files | Lines | ||
---|---|---|---|---|---|---|
... | ||||||
* | useless-super-delegation takes into account default values for the current ↵ | hippo91 | 2017-09-30 | 1 | -0/+96 | |
| | | | | | implementation A method can reimplement a super method in order to provide a different default value, in which case we shouldn't emit ``useless-super-delegation``. | |||||
* | Skip the invalid-length-returned check when the return value is clearly an ↵ | Claudiu Popa | 2017-09-29 | 1 | -1/+8 | |
| | | | | int, but with an uninferable underlying value | |||||
* | Adding a check for ``inconsistent-return-statements`` inside function or ↵ | hippo91 | 2017-09-26 | 1 | -4/+4 | |
| | | | | | methods. (#1641) Close #1267 | |||||
* | Add a couple of Uninferable filters where we weren't using any | Claudiu Popa | 2017-08-15 | 1 | -2/+2 | |
| | ||||||
* | Fix line-too-long | Claudiu Popa | 2017-06-03 | 1 | -1/+2 | |
| | ||||||
* | Don't emit assigning-non-slot when reassigning the class with another class ↵ | Claudiu Popa | 2017-06-02 | 1 | -0/+12 | |
| | | | | that has the same layout. Close #1506 | |||||
* | Remove mentions of deprecated astroid node names (#1485) | Łukasz Rogalski | 2017-05-22 | 1 | -4/+4 | |
| | ||||||
* | Do not emit warning on type(self)._private_attribute access (#1288) | Łukasz Rogalski | 2017-02-05 | 1 | -7/+26 | |
| | | | Closes #1031 | |||||
* | Change the text of arguments-differ by making it more explicit | Claudiu Popa | 2016-12-30 | 1 | -1/+1 | |
| | ||||||
* | Take in consideration the dummy-rgx-variable when checking for different ↵ | Claudiu Popa | 2016-12-30 | 1 | -16/+32 | |
| | | | | parameters. Close #1041 | |||||
* | Fix formatting and use any() to simplify the check | Claudiu Popa | 2016-12-18 | 1 | -12/+22 | |
| | ||||||
* | Add new checker len-as-condition (#1154) | Alexander Todorov | 2016-11-29 | 1 | -1/+1 | |
| | | | | | | * Add new checker len-as-condition * Fix len(SEQUENCE) warnings discovered by new checker | |||||
* | Report when __slots__ is a simple string and not a non-string container (#1171) | Anthony Foglia | 2016-11-24 | 1 | -0/+5 | |
| | | | | | | * Report when __slots__ is a simple string and not a non-string container * Update ChangeLog | |||||
* | Don't try to access variables defined in a separate scope when checking for ↵ | Claudiu Popa | 2016-11-20 | 1 | -7/+26 | |
| | | | | protected-access. Close #1126 | |||||
* | Grammar fix | Claudiu Popa | 2016-07-23 | 1 | -1/+1 | |
| | ||||||
* | Exempt special methods from arguments-differ. Close #1042 | Claudiu Popa | 2016-07-23 | 1 | -1/+7 | |
| | ||||||
* | Do not emit useless-super-delegation when having extraneous arguments in the ↵ | Claudiu Popa | 2016-07-23 | 1 | -1/+25 | |
| | | | | call site. Close #1043 | |||||
* | Use ClassDef.instantiate_class() (#1040) | Florian Bruhin | 2016-07-23 | 1 | -1/+1 | |
| | | | | | | | | Otherwise we get this from astroid: .../pylint/pylint/checkers/classes.py:1112: PendingDeprecationWarning: ClassDef.instanciate_class() is deprecated and slated for removal in astroid 2.0, use ClassDef.instantiate_class() instead. instance = cls.instanciate_class() | |||||
* | Even more granular copyrights (thanks to copyrite) | Claudiu Popa | 2016-07-23 | 1 | -2/+8 | |
| | ||||||
* | Keep a consistent copyright notice across the board. | Claudiu Popa | 2016-07-19 | 1 | -0/+2 | |
| | | | | | This was changed automatically in #894, but apparently we need to have the copyright notice somewhere. | |||||
* | Do not check for useless-super-delegation if the function is decorated | Moises Lopez - https://www.vauxoo.com/ | 2016-07-17 | 1 | -0/+4 | |
| | ||||||
* | Uninferable now has a false default value, so don't check it explicitly. | Claudiu Popa | 2016-07-16 | 1 | -1/+1 | |
| | ||||||
* | Add a new warning, 'useless-super-delegation' | Claudiu Popa | 2016-07-15 | 1 | -1/+135 | |
| | | | | | | | | | | This is used whenever pylint can detect than an overridden method is useless, relying on super() delegation to implement the same thing as another method from MRO. In this case, it is enough to not implement the given method and let it be propagated to the another implementation from the MRO. Close #839 | |||||
* | arguments-differ takes in consideration kwonlyargs and variadics | Claudiu Popa | 2016-07-10 | 1 | -6/+16 | |
| | | | | | | | | | | | | | | | | | | | | The check was rewritten to be more strict about overridding a method: - now it complains if the parent method had variadics in the signature, but the overridden method does not have. For instance, having *args in the parent method means that it can accept anything, but restricting this to (a, b) in the overridden method leads to inconsistencies in their behavior and can potentially introduce new bugs. - it complains if the number of keyword only parameters are not the same in the overridden method - it complains about the names of the parameters. For instance, having (a, b) in the parent method and changing this to (c, d) in the overridden method means that the latter one cannot be accessed in the same way as the parent method, when using keyword arguments. Close #983 | |||||
* | Do not complain about arguments-differ if the overridden method has variadic ↵ | Claudiu Popa | 2016-07-10 | 1 | -0/+4 | |
| | | | | positional arguments. | |||||
* | Check the names of the parameters when checking for overridden methods with ↵ | Claudiu Popa | 2016-07-10 | 1 | -10/+36 | |
| | | | | different parameters | |||||
* | Fix bad indentation. | Claudiu Popa | 2016-07-06 | 1 | -2/+1 | |
| | ||||||
* | Remove features which were scheduled for removal. | Claudiu Popa | 2016-07-06 | 1 | -7/+2 | |
| | ||||||
* | Add the new shorter license header, including to missing files. Close #894. | Claudiu Popa | 2016-06-01 | 1 | -15/+3 | |
| | ||||||
* | Fix typos (#916) | Jakub Wilk | 2016-05-25 | 1 | -2/+2 | |
| | | | | | | | | * Split words that were inadvertently glued together * Fix typos in the documentation * Fix typos in message descriptions | |||||
* | Don't crash when checking, for super-init-not-called, a method defined in an ↵ | Claudiu Popa | 2016-05-10 | 1 | -1/+2 | |
| | | | | if block. | |||||
* | Filter non Const nodes instead of checking for AttributeError. | Claudiu Popa | 2016-03-16 | 1 | -6/+8 | |
| | ||||||
* | New check: invalid-length-returned | Łukasz Rogalski | 2016-03-14 | 1 | -1/+18 | |
| | | | | Implementation of issue #557. | |||||
* | remove @check_message decorator from [visit|leave]_classdef methods of the ↵ | Sylvain Thenault | 2016-01-19 | 1 | -7/+0 | |
| | | | | | | | | | | | ClassChecker Those methods are actually not only emitting the listed messages but also updating some internal state that is used from other, unrestricted, messages. We should work on separating responsabilities if we want to restrict method entries upon messages restriction. Closes issue #789 | |||||
* | Catch more cases as not proper iterables for __slots__ | Claudiu Popa | 2016-01-18 | 1 | -9/+4 | |
| | | | | Close issue #775 | |||||
* | Add new missing messages from check_messages decorator. | Claudiu Popa | 2016-01-15 | 1 | -1/+3 | |
| | ||||||
* | Mark missing messages with check_messages. | Claudiu Popa | 2016-01-15 | 1 | -2/+3 | |
| | ||||||
* | Treat AsyncFunctionDef just like FunctionDef nodes | Claudiu Popa | 2016-01-09 | 1 | -0/+4 | |
| | | | | | | | | | When support for Python 3.5 was added, AsyncFunctionDef wasn't handled properly in terms of FunctionDef, which meant that most of the checks which involved a function were never called for AsyncFunctionDef. This led to spurious false positives which occurred when AsyncFunctionDef were analyzed. Closes #767 | |||||
* | Fix new import related errors in pylint's codebase. | Claudiu Popa | 2015-11-25 | 1 | -2/+3 | |
| | ||||||
* | Fix instances of unneeded-not in pylint's codebase. | Claudiu Popa | 2015-11-25 | 1 | -1/+1 | |
| | ||||||
* | Cleanup pylint issues | Claudiu Popa | 2015-11-21 | 1 | -9/+6 | |
| | | | | | | | | | | | 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. | |||||
* | Import has_known_bases and safe_infer back into pylint from astroid, until ↵ | Claudiu Popa | 2015-11-20 | 1 | -5/+5 | |
| | | | | | | | | | 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. | |||||
* | Refactor non-iterator-returned checker to use safe inference | Dmitry Pribysh | 2015-11-09 | 1 | -8/+24 | |
| | | | | This way it won't infer all possibles values of a node, only two at most. | |||||
* | Reduce number of false positives emitted by non-iterator-returned checker | Dmitry Pribysh | 2015-11-07 | 1 | -4/+7 | |
| | | | | | | | | 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. | |||||
* | Separe the conditions to improve the readability of the code. | Claudiu Popa | 2015-10-30 | 1 | -0/+4 | |
| | ||||||
* | Prefer any instead of for. | Claudiu Popa | 2015-10-30 | 1 | -4/+3 | |
| | ||||||
* | Merged in lmedioni/pylint (pull request #295) | Claudiu Popa | 2015-10-30 | 1 | -3/+43 | |
|\ | | | | | | | check for class methods declared without a decorator | |||||
| * | no-static/class-method: enhance the tests and fix the code accordingly | Laura Médioni | 2015-10-29 | 1 | -4/+3 | |
| | | ||||||
| * | improve style and fix typos regarding no_class/staticmethod_decorator | Laura Médioni | 2015-10-29 | 1 | -2/+3 | |
| | | ||||||
| * | check for static methods declared without a decorator | Laura Médioni | 2015-10-29 | 1 | -5/+12 | |
| | | | | | | | | closes issue #675 |