Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | 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 | ||||
| * | check for class methods declared without a decorator | Laura M?dioni | 2015-10-29 | 1 | -3/+36 |
| | | | | | | | | related to the issue #675 | ||||
* | | Make iterable checker skip classes that are inferred to be abstractfix-685 | Dmitry Pribysh | 2015-10-27 | 1 | -10/+1 |
|/ | |||||
* | non-iterator-returned can detect classes with iterator-metaclasses. | Claudiu Popa | 2015-10-20 | 1 | -11/+27 |
| | | | | Closes issue #679. | ||||
* | Rename checking function for 'super()' call checking and update Changelog.fix-667 | Dmitry Pribysh | 2015-10-18 | 1 | -2/+2 |
| | |||||
* | Make `no-self-use` checker not throw a warning if method has a `super()` call. | Dmitry Pribysh | 2015-10-17 | 1 | -1/+10 |
| | | | | | | | This is only enabled for python 3.0+ and only for `super` calls with no arguments. Fixes issue #667. | ||||
* | Don't emit 'assigning-non-slot' for descriptors. Closes issue #652. | Claudiu Popa | 2015-10-04 | 1 | -0/+23 |
| | |||||
* | Use the new node names for the visit and leave methods | Claudiu Popa | 2015-08-26 | 1 | -7/+7 |
| | | | | | Also, emit a PendingDeprecationWarning if the old names are still used, this support being removed in pylint 2.0. | ||||
* | Update pylint to use the new astroid AST names and methods | Claudiu Popa | 2015-08-25 | 1 | -30/+28 |
| | | | | | | | Also improve a couple of imports and added support for the old visit_ names. Priority will have the old visit_ names, such as visit_discard over visit_expr, visit_callfunc over visit_call etc. | ||||
* | Don't emit no-init if not all the bases from a class are known. | Claudiu Popa | 2015-07-22 | 1 | -1/+1 |
| | | | | Closes issue #604. | ||||
* | Use safe_infer and has_known_bases from astroid.helpers. Closes issue #593. | Claudiu Popa | 2015-07-06 | 1 | -3/+4 |
| | |||||
* | Detect a couple of objects which can't be base classes (bool, slice, range ↵ | Claudiu Popa | 2015-06-25 | 1 | -1/+8 |
| | | | | and memoryview, which weren't detected until now). | ||||
* | Fix a false positive regarding staticmethods: they don't have an implicit ↵ | Claudiu Popa | 2015-06-15 | 1 | -2/+1 |
| | | | | argument when 'bounded'. | ||||
* | Remove trailing whitespace. | Claudiu Popa | 2015-06-14 | 1 | -3/+3 |
| | |||||
* | arguments-differ understand differences between various type of functions. | Claudiu Popa | 2015-06-14 | 1 | -6/+32 |
| | | | | | This patch improves the way how arguments-differ understands the differences between classmethods, staticmethods and properties. Closes issue #548. | ||||
* | Pylint understand super calls. | Claudiu Popa | 2015-05-28 | 1 | -0/+3 |
| | |||||
* | Fix some pylint warnings over pylint's codebase. | Claudiu Popa | 2015-05-16 | 1 | -3/+3 |
| | |||||
* | Remove trailing whitespaces. | Claudiu Popa | 2015-05-14 | 1 | -3/+3 |
| | |||||
* | Remove 'bad-context-manager' due to the inclusion of ↵ | Claudiu Popa | 2015-05-14 | 1 | -19/+2 |
| | | | | 'unexpected-special-method-signature'. | ||||
* | Add a new warning 'unexpected-special-method-signature'. | Claudiu Popa | 2015-05-14 | 1 | -3/+58 |
| | | | | | This is emitted when a special method (dunder method) doesn't have the expected signature, which can lead to actual errors in the application code. Closes issue #253. | ||||
* | Don't emit attribute-defined-outside-init and ↵ | Claudiu Popa | 2015-05-12 | 1 | -1/+9 |
| | | | | | | | | access-member-before-definition for mixin classes. Actual errors can occur in mixin classes, but this is controlled by the ignore-mixin-members option, so if something is a legitimate error, then the user can pass --ignore-mixin-members=no in order to check these in mixin classes as well. Closes issue #412. | ||||
* | Cleanup. | Claudiu Popa | 2015-05-11 | 1 | -6/+6 |
| | |||||
* | Move non-iterator-returned to a new checker, SpecialMethodsChecker, which ↵ | Claudiu Popa | 2015-05-10 | 1 | -26/+46 |
| | | | | has checks for structural checking of special methods. | ||||
* | Remove unused imports. | Claudiu Popa | 2015-05-07 | 1 | -1/+0 |
| | |||||
* | Make ignore-iface-methods a noop option, mark it as deprecated. | Claudiu Popa | 2015-05-05 | 1 | -15/+5 |
| | |||||
* | Remove the rest of interface checks. | Claudiu Popa | 2015-05-05 | 1 | -74/+4 |
| | | | | | | | The removed checks are interface-is-not-class, missing-interface-method, unresolved-interface. The reason is that its better to start recommending ABCs instead of the old Zope era of interfaces. | ||||
* | Add two new warnings, duplicate-bases and inconsistent-mro. | Claudiu Popa | 2015-05-03 | 1 | -1/+21 |
| | | | | | | duplicate-bases is emitted when a class has the same bases listed more than once in its bases definition, while inconsistent-mro is emitted when no sane mro hierarchy can be determined. Closes issue #526. | ||||
* | Don't warn about no-self-use for builtin properties. | Claudiu Popa | 2015-04-06 | 1 | -1/+2 |
| | |||||
* | Add Changelog entry and cleanup after changeset 2307df9 | Claudiu Popa | 2015-03-31 | 1 | -2/+0 |
| | |||||
* | Promote a couple of warnings to errors. | Claudiu Popa | 2015-03-20 | 1 | -2/+3 |
| | | | | | | | | These warnings were promoted since they could uncover potential bugs in the code and since most people are using `pylint -E` anyway, it's good to have them as errors. These warnings are: assignment-from-none, unbalanced-tuple-unpacking, unpacking-non-sequence, non-iterator-returned. Closes issue #388. | ||||
* | Fix a couple of doc warnings, use string continuation. | Claudiu Popa | 2015-03-02 | 1 | -43/+43 |
| | |||||
* | Move all package files to a pylint package. | Ionel Cristian Maries | 2015-02-14 | 1 | -0/+982 |