Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Move all package files to a pylint package. | Ionel Cristian Maries | 2015-02-14 | 568 | -15757/+0 |
| | |||||
* | Add warning for "unichr" in Python 3 | Cosmin Poieana | 2015-02-14 | 1 | -0/+1 |
| | | | | Closes issue #472 | ||||
* | Add warning for "intern" in Python 3 | Viorel Stirbu | 2015-02-14 | 1 | -0/+1 |
| | | | | Closes issue #473 | ||||
* | Use all the inferred statements for the super-init-not-called check. | Claudiu Popa | 2015-02-09 | 1 | -0/+16 |
| | | | | | | | | | 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. | ||||
* | Add test for issue #308. Closes issue #308. | Claudiu Popa | 2015-02-09 | 1 | -0/+16 |
| | |||||
* | Merged in mihai_stan/pylint (pull request #217) | Claudiu Popa | 2015-02-05 | 1 | -0/+12 |
|\ | | | | | | | Catch enchant check exceptions. Closes issue #469. | ||||
| * | Added C0403 invalid-characters-in-docstring for docstrings with invalid ↵ | mihai_stan | 2015-02-05 | 1 | -0/+12 |
| | | | | | | | | characters (e.g. \x00) | ||||
* | | Look all the methods from ancestors for too-few-public-methods. | Claudiu Popa | 2015-02-05 | 1 | -0/+16 |
|/ | | | | | | | | | 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 Popa | 2015-01-30 | 1 | -0/+10 |
| | | | | | | | | | 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 Popa | 2015-01-24 | 2 | -2/+6 |
| | |||||
* | Fix a crash which occurred when using multiple jobs and the files | Claudiu Popa | 2015-01-16 | 1 | -1/+3 |
| | | | | given as argument didn't exist at all. | ||||
* | Check the number of arguments before attempting to obtain the first one. | Claudiu Popa | 2015-01-12 | 1 | -0/+8 |
| | |||||
* | Add the method names for the 'signature-differs' and 'argument-differs' | Claudiu Popa | 2015-01-11 | 2 | -2/+2 |
| | | | | | | warnings. Closes issue #433. | ||||
* | Add ChangeLog entry for the JSON report and fix the test. | Claudiu Popa | 2015-01-04 | 1 | -11/+18 |
| | |||||
* | Merged in vtemian/pylint (pull request #198) | Claudiu Popa | 2015-01-04 | 2 | -1/+57 |
|\ | | | | | | | Added json reporter #ropython-sprints | ||||
| * | Merged from remote | Vlad Temian | 2014-11-22 | 48 | -283/+366 |
| |\ | |||||
| * | | Added JSONReporter to test_self.py.RunTC.test_all | Vlad Temian | 2014-11-22 | 1 | -1/+3 |
| | | | |||||
| * | | Added json reporter | Vlad Temian | 2014-11-04 | 1 | -0/+54 |
| | | | |||||
* | | | Improve a couple of things for redundant-unittest-assert. | Claudiu Popa | 2015-01-04 | 2 | -4/+13 |
| | | | | | | | | | | | | | | | | | | 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 Popa | 2015-01-04 | 2 | -0/+27 |
|\ \ \ | | | | | | | | | | | | | Added check for redundant assert #ropython-sprints | ||||
| * | | | Improve testsredundant-assert | Vlad Temian | 2014-11-22 | 2 | -0/+6 |
| | | | | |||||
| * | | | Fix code style and naming convention issues after review | Vlad Temian | 2014-11-22 | 2 | -4/+12 |
| | | | | |||||
| * | | | Test redundant unittest assert | Vlad Temian | 2014-11-22 | 2 | -0/+13 |
| | |/ | |/| | |||||
* | | | Fix a false positive with invalid-slots-objects and unicode strings. Closes ↵ | Claudiu Popa | 2015-01-03 | 1 | -0/+12 |
| | | | | | | | | | | | | issue #421. | ||||
* | | | Use a DeprecationWarning for ParseableTextReporter. | Claudiu Popa | 2015-01-03 | 1 | -1/+12 |
| | | | |||||
* | | | Fix a false positive on Python 2 for raising-bad-type and tuples. | Claudiu Popa | 2015-01-02 | 1 | -0/+61 |
| | | | | | | | | | | | | | | | Raising tuples in the form 'raise (ZeroDivisionError, None)' is not an error on Python 2, but it is an error on Python 3. | ||||
* | | | Cleanup lint.py. | Claudiu Popa | 2015-01-02 | 1 | -2/+2 |
| | | | | | | | | | | | | | | | Import modules, instead of objects from them, used only once. Trim a couple of long lines, remove a couple of unused variables and so on. | ||||
* | | | Don't filter out builtins when checking for raising-non-exception. | Claudiu Popa | 2015-01-02 | 2 | -10/+39 |
| | | | | | | | | | | | | | | | There is no reason anymore to exclude builtins for nonstandard-exception and raising-non-exception. | ||||
* | | | Fix a false negative with raising-non-exception. | Claudiu Popa | 2015-01-02 | 3 | -0/+16 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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 Popa | 2015-01-02 | 3 | -9/+9 |
| | | | |||||
* | | | Fix a false positive with catching-non-exception and tuples of | Claudiu Popa | 2014-12-29 | 2 | -1/+23 |
| | | | | | | | | | | | | exceptions. | ||||
* | | | Use a mro traversal for finding abstract methods. Closes issue #415. | Claudiu Popa | 2014-12-21 | 8 | -40/+216 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | 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. | ||||
* | | | Fix typos in docstringBioGeek/fix-typos-in-docstring-1418910993320 | BioGeek | 2014-12-18 | 1 | -1/+1 |
| | | | |||||
* | | | Proper abstract method lookup while checking for abstract-class-instantiated. | Claudiu Popa | 2014-12-16 | 2 | -3/+54 |
| | | | | | | | | | | | | | | | This depends on the new Class.mro method, which will be added in astroid 1.3.3. Closes issue #401. | ||||
* | | | Have the python3 checker warn for round() and input(). | Brett Cannon | 2014-12-12 | 1 | -1/+2 |
| | | | | | | | | | | | | | | | | | | | | | Both built-ins have changed semantics in Python 3 that you cannot make compatible inline. Closes issue #411. | ||||
* | | | Fix 2.x open mode check; Add more tests | Cosmin Poieana | 2014-12-05 | 4 | -0/+34 |
| | | | |||||
* | | | Merge; Fix open mode bug | Cosmin Poieana | 2014-12-05 | 6 | -5/+35 |
|\ \ \ | |||||
| * \ \ | Merge; Fix according to review | Cosmin Poieana | 2014-11-30 | 6 | -5/+31 |
| |\ \ \ | |||||
| | * \ \ | Fix opening mode check. Closes Issue #307 | Cosmin Poieana | 2014-11-22 | 6 | -5/+26 |
| | |\ \ \ | | | |/ / | |||||
* | | | | | Check the return of properties when checking for not-callable. | Claudiu Popa | 2014-12-05 | 1 | -1/+46 |
| | | | | | | | | | | | | | | | | | | | | Closes issue #406. | ||||
* | | | | | Simplify _annotated_unpack_infer. | Claudiu Popa | 2014-12-04 | 2 | -0/+9 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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. | ||||
* | | | | | Improve the reporting for dangerous-default-value. | Claudiu Popa | 2014-12-03 | 3 | -4/+20 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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 Popa | 2014-12-03 | 2 | -0/+17 |
|/ / / / | |||||
* | | | | Added test for issue #263 | Calin Don | 2014-11-22 | 1 | -0/+47 |
|/ / / | |||||
* | | | Convert some testcases to the new framework. | Torsten Marek | 2014-11-22 | 18 | -70/+64 |
| | | | |||||
* | | | Use the public method astroid.nodes.Class.is_subtype_of. | Torsten Marek | 2014-11-22 | 1 | -1/+1 |
| | | | |||||
* | | | Fix tests to work with the new astroid release. | Claudiu Popa | 2014-11-22 | 5 | -9/+3 |
| | | | |||||
* | | | Fix false positives with the old-octal-literal. | Claudiu Popa | 2014-11-22 | 1 | -1/+1 |
|/ / | |||||
* | | tests: fix more tests broken by max-line-length change | Sylvain Th?nault | 2014-11-19 | 2 | -2/+2 |
| | | |||||
* | | Change default max-line-length to 100 rather than 80 | Sylvain Th?nault | 2014-11-19 | 5 | -17/+17 |
| | |