summaryrefslogtreecommitdiff
path: root/checkers
Commit message (Collapse)AuthorAgeFilesLines
* Move all package files to a pylint package.Ionel Cristian Maries2015-02-1419-9257/+0
|
* Fix char typounichrCosmin Poieana2015-02-141-1/+1
|
* Add warning for "unichr" in Python 3Cosmin Poieana2015-02-141-0/+6
| | | | Closes issue #472
* Add warning for "intern" in Python 3Viorel Stirbu2015-02-141-0/+6
| | | | Closes issue #473
* Use all the inferred statements for the super-init-not-called check.Claudiu Popa2015-02-091-20/+21
| | | | | | | | | For the super-init-not-called check, ``next(expr.infer())`` was used, which returned as the first statement an YES node, which resulted in the node being skipped for processing. Inferring all the objects would have returned the actual classes for which the parent was indeed called. This patch changes that to a infer-all strategy, which should yield better results. Closes issue #389.
* Merged in mihai_stan/pylint (pull request #217)Claudiu Popa2015-02-051-1/+10
|\ | | | | | | Catch enchant check exceptions. Closes issue #469.
| * Added C0403 invalid-characters-in-docstring for docstrings with invalid ↵mihai_stan2015-02-051-3/+10
| | | | | | | | characters (e.g. \x00)
| * Treat nulls as separatorsmihai_stan2015-02-051-0/+2
| |
* | Look all the methods from ancestors for too-few-public-methods.Claudiu Popa2015-02-051-14/+21
|/ | | | | | | | | This patch uses two counters for too-few-public-methods and too-many-public-methods. In order to avoid false positives, too-many-public-methods check is looking only for methods defined in the current class. On the other hand, too-few-public-methods should look in ancestors as well, since most of the time this check is useful only for simple classes, without many public methods. Closes issue #471.
* Fix a false positive with `too-few-format-args` warning.Claudiu Popa2015-01-301-1/+1
| | | | | | | | | The warning was emitted when the string format contained a normal positional argument ('{0}'), mixed with a positional argument which did an attribute access ('{0.__class__}'). Closes issue #463.
* Don't require a docstring for empty modules. Closes issue #261.Claudiu Popa2015-01-241-0/+5
|
* Suppress ResolveErrors from Class.mro in unimplemented_abstract_methods.Claudiu Popa2015-01-151-0/+4
|
* Check the number of arguments before attempting to obtain the first one.Claudiu Popa2015-01-121-1/+1
|
* Add the method names for the 'signature-differs' and 'argument-differs'Claudiu Popa2015-01-111-4/+8
| | | | | | warnings. Closes issue #433.
* Improve a couple of things for redundant-unittest-assert.Claudiu Popa2015-01-041-3/+3
| | | | | | Include the bad assert method in the message and improve the test coverage. Also, annotate the test class with a skip decorator, since unittest discovery will believe it's an actual unittest file.
* Merged in vtemian/pylint/redundant-assert (pull request #202)Claudiu Popa2015-01-041-6/+24
|\ | | | | | | Added check for redundant assert #ropython-sprints
| * Improve testsredundant-assertVlad Temian2014-11-221-4/+4
| |
| * Fix code style and naming convention issues after reviewVlad Temian2014-11-221-7/+10
| |
| * Added check for redundant assert in unittestVlad Temian2014-11-221-6/+21
| |
* | Fix a false positive with invalid-slots-objects and unicode strings. Closes ↵Claudiu Popa2015-01-031-1/+1
| | | | | | | | issue #421.
* | Close the streams in similar checker.Claudiu Popa2015-01-031-2/+6
| |
* | Use the new Module.stream, since Module.file_stream is deprecated.Claudiu Popa2015-01-032-8/+6
| |
* | Fix a false positive on Python 2 for raising-bad-type and tuples.Claudiu Popa2015-01-021-3/+22
| | | | | | | | | | Raising tuples in the form 'raise (ZeroDivisionError, None)' is not an error on Python 2, but it is an error on Python 3.
* | Don't filter out builtins when checking for raising-non-exception.Claudiu Popa2015-01-021-2/+1
| | | | | | | | | | There is no reason anymore to exclude builtins for nonstandard-exception and raising-non-exception.
* | Fix a false negative with raising-non-exception.Claudiu Popa2015-01-021-11/+19
| | | | | | | | | | | | | | | | | | | | When the raise used an uninferrable exception context, as in the following example, raising-non-exception wouldn't have been emitted, due to a misplaced return. The example is: from missing_import import something raise SomeClass from something
* | Add the name of the abstract class in abstract-class-instantiated message.Claudiu Popa2015-01-021-3/+7
| |
* | Small cleanup to exceptions checkerClaudiu Popa2015-01-021-4/+3
| |
* | The comment is not true anymore.Claudiu Popa2015-01-021-5/+0
| |
* | Small refactoring to exceptions checker.Claudiu Popa2015-01-021-13/+24
| |
* | Fix a false positive with catching-non-exception and tuples ofClaudiu Popa2014-12-291-26/+43
| | | | | | | | exceptions.
* | Use a mro traversal for finding abstract methods. Closes issue #415.Claudiu Popa2014-12-213-53/+81
| | | | | | | | | | | | | | | | | | This patch adds a new unimplemented_abstract_methods in pylint.checkers.utils, which is used to obtain all the abstract methods which weren't implemented anywhere in the mro of a class. The code works now by traversing the mro, gathering all abstract methods encountered at each step and resolving the implemented ones through either definition or assignment. This disables a couple of false positives on classes with complex hierarchies.
* | Proper abstract method lookup while checking for abstract-class-instantiated.Claudiu Popa2014-12-161-2/+31
| | | | | | | | | | This depends on the new Class.mro method, which will be added in astroid 1.3.3. Closes issue #401.
* | Fix typos in error messages.Claudiu Popa2014-12-151-2/+2
| |
* | Have the python3 checker warn for round() and input().Brett Cannon2014-12-121-3/+15
| | | | | | | | | | | | | | Both built-ins have changed semantics in Python 3 that you cannot make compatible inline. Closes issue #411.
* | Minimize the except block, by catching only what will raise NoSuchArgumentError.Claudiu Popa2014-12-061-9/+11
| |
* | Mergeopen_modeCosmin Poieana2014-12-061-0/+2
|\ \
| * | Check the return value of slots, it can be None.Claudiu Popa2014-12-051-0/+2
| | |
* | | Fix 2.x open mode check; Add more testsCosmin Poieana2014-12-051-2/+9
| | |
* | | Merge; Fix open mode bugCosmin Poieana2014-12-051-3/+41
|\ \ \ | |/ / |/| |
| * | Merge; Fix according to reviewCosmin Poieana2014-11-301-3/+41
| |\ \
| | * \ Fix opening mode check. Closes Issue #307Cosmin Poieana2014-11-221-2/+42
| | |\ \
| | | * | Proper opening mode checking for filesCosmin Poieana2014-11-221-2/+42
| | | |/
* | | | Check the return of properties when checking for not-callable.Claudiu Popa2014-12-051-4/+9
| | | | | | | | | | | | | | | | Closes issue #406.
* | | | Simplify _annotated_unpack_infer.Claudiu Popa2014-12-041-12/+5
| | | | | | | | | | | | | | | | | | | | | | | | With this fix, _annotated_unpack_infer doesn't try to unpack everything, doing only a linear inference and unpacking. This also removes the false positives where a part of the except handler is a builtin call for list/tuple/set.
* | | | Simplify the code by using safe_infer.Claudiu Popa2014-12-041-14/+10
| | | |
* | | | Make some methods private.Claudiu Popa2014-12-041-4/+4
| | | |
* | | | Improve the reporting for dangerous-default-value.Claudiu Popa2014-12-031-9/+21
| | | | | | | | | | | | | | | | | | | | | | | | The new version takes in consideration the fact that a couple of builtins, dict, tuple, set and list can be inferred to their AST syntax node, which messed up the reporting for dangerous-default-value until now.
* | | | Look only in the current function's scope for bad-super-call. Closes issue #403.Claudiu Popa2014-12-031-1/+8
|/ / /
* | | Make dangerous-default-argument check resilient to inferred builtin calls ↵Claudiu Popa2014-11-281-1/+4
| | | | | | | | | | | | for list/tuple/set.
* | | fixed reading list of ignored words for spelling checkerissue-398Michal Nowikowski2014-11-271-1/+1
|/ /
View Lorry Controller Status