Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Merge with defaultiterable-checker | Dmitry Pribysh | 2015-10-18 | 22 | -1/+473 |
|\ | |||||
| * | Make iterable/mapping checker more smart and refactor it. | Dmitry Pribysh | 2015-10-15 | 17 | -81/+211 |
| | | | | | | | | | | Now it ignores errors inside mixins declarations and is able to recognize iterable/mapping metaclasses. | ||||
| * | Remove redundant unittests for iterable checker | Dmitry Pribysh | 2015-10-08 | 1 | -135/+0 |
| | | |||||
| * | Handle more corner cases in iterable/mapping checker | Dmitry Pribysh | 2015-10-08 | 5 | -33/+109 |
| | | |||||
| * | Add more positive functional tests for the iterable context checker | Dmitry Pribysh | 2015-10-08 | 2 | -25/+79 |
| | | |||||
| * | Refactor iterable context checker | Dmitry Pribysh | 2015-10-08 | 1 | -29/+57 |
| | | |||||
| * | Move iterable checker to typecheck module and alter inference strategy | Dmitry Pribysh | 2015-10-08 | 4 | -223/+264 |
| | | |||||
| * | Add functional tests for iterable hecker | Dmitry Pribysh | 2015-10-07 | 5 | -0/+56 |
| | | |||||
| * | Alter inference strategy in iterable/mapping checker | Dmitry Pribysh | 2015-10-07 | 2 | -52/+22 |
| | | |||||
| * | Improve iterable inference and add more unit tests for iterable checker | Dmitry Pribysh | 2015-10-06 | 2 | -11/+62 |
| | | |||||
| * | Add iterable checking for all types of comprehensions. | Dmitry Pribysh | 2015-10-05 | 2 | -0/+36 |
| | | | | | | | | | | Specifically, list, dict and set comprehensions along with generator expressions. | ||||
| * | Add initial version of checker for iterables/mappings. | Dmitry Pribysh | 2015-10-05 | 2 | -0/+165 |
| | | | | | | | | | | | | | | | | | | | | | | | | It checks for the following things: - for-statement should contain an iterable value - `yield from`-statement should contain an iterable value - function call with star args should contain iterable value (e.g. in `foo(*bar)` bar should be an iterable) - function call with kwargs should contain a mapping (e.g. in `foo(**bar)` bar should be a dict) Idea came from issue #563. | ||||
* | | Forgot to add test file. | Claudiu Popa | 2015-10-18 | 1 | -0/+5 |
| | | |||||
* | | Merged in dmand/pylint/fix-667 (pull request #287) | Claudiu Popa | 2015-10-18 | 4 | -1/+25 |
|\ \ | | | | | | | | | | Make `no-self-use` checker not throw a warning if method has a `super()` call. | ||||
| * | | 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 | 4 | -1/+25 |
| |/ | | | | | | | | | | | | | This is only enabled for python 3.0+ and only for `super` calls with no arguments. Fixes issue #667. | ||||
* | | Decode the docstring before attempting to parse it with the spelling checker | Claudiu Popa | 2015-10-16 | 6 | -13/+21 |
| | | | | | | | | | | Also, the patch changes calls to str() in the list of html reporter's messages only for the objects which aren't already unicode or byte strings. | ||||
* | | Simplify compute_content, by removing the custom implementations of write ↵ | Claudiu Popa | 2015-10-16 | 1 | -26/+11 |
| | | | | | | | | and writeln. | ||||
* | | Update the docstrings to be more intuitive. | Claudiu Popa | 2015-10-16 | 1 | -4/+2 |
| | | |||||
* | | Remove commented out code. | Claudiu Popa | 2015-10-16 | 1 | -2/+0 |
| | | |||||
* | | Make encode a proper method of the BaseReporter, which removes the need of ↵ | Claudiu Popa | 2015-10-16 | 1 | -7/+4 |
| | | | | | | | | patching it all the time. | ||||
* | | Remove handling of pending_urls, which is unused by pylint. | Claudiu Popa | 2015-10-16 | 1 | -6/+0 |
| | | |||||
* | | Remove BaseComponent and additional attributes and methods | Claudiu Popa | 2015-10-16 | 3 | -44/+7 |
| | | | | | | | | | | The BaseComponent's arguments weren't actually used and by removing them, a bunch of other methods were removed as well. | ||||
* | | Use unicode string. | Claudiu Popa | 2015-10-16 | 1 | -1/+1 |
| | | |||||
* | | Move VNode into ureports.nodes. | Claudiu Popa | 2015-10-16 | 2 | -58/+40 |
| | | |||||
* | | Remove unused APIs from VNode class, which got merged with Node class. | Claudiu Popa | 2015-10-16 | 1 | -182/+4 |
| | | |||||
* | | Simplify the code a little bit by using enumerate instead of a range and len ↵ | Claudiu Popa | 2015-10-15 | 1 | -5/+4 |
| | | | | | | | | combination. | ||||
* | | Use the u'' prefix throught text writer. | Claudiu Popa | 2015-10-15 | 1 | -1/+1 |
| | | |||||
* | | Use enumerate instead of range. | Claudiu Popa | 2015-10-15 | 1 | -2/+1 |
| | | |||||
* | | Rewrite the operation with a ternary if in order to be clearer. | Claudiu Popa | 2015-10-15 | 1 | -1/+1 |
| | | |||||
* | | Remove unused nodes and corresponding visit methods. | Claudiu Popa | 2015-10-15 | 3 | -64/+0 |
| | | |||||
* | | Port more old tests to the new style tests and remove obsolete tests. | Claudiu Popa | 2015-10-12 | 17 | -98/+83 |
| | | |||||
* | | Update to use the new location of NodeNG. | Claudiu Popa | 2015-10-12 | 1 | -2/+2 |
| | | |||||
* | | Rename the variable to fix an undefined-loop-variable false positive. | Claudiu Popa | 2015-10-10 | 1 | -2/+2 |
| | | |||||
* | | --enable=all can now be used. Closes issue #142. | Claudiu Popa | 2015-10-10 | 2 | -0/+20 |
| | | |||||
* | | Don't warn about Starred nodes used properly in unpacking contexts | Claudiu Popa | 2015-10-10 | 3 | -0/+12 |
| | | | | | | | | Closes issue #653 | ||||
* | | Use the any builtin, instead of evaluating the entire list of instance ↵ | Claudiu Popa | 2015-10-09 | 1 | -1/+1 |
| | | | | | | | | attributes. | ||||
* | | Add a new error, 'repeated-keyword', when a keyword argument is passed ↵ | Claudiu Popa | 2015-10-09 | 7 | -6/+82 |
| | | | | | | | | | | | | | | | | | | | | multiple times into a function call. This is similar with redundant-keyword-arg, but it's mildly different that it needs to be a separate error. This change also uses a CallSite for understanding the arguments that were passed into a function call, since with this we can make sense about multiple starred arguments passed into (PEP 448). | ||||
* | | Remove the starargs verification code, since it is obsolete after the AST ↵ | Claudiu Popa | 2015-10-07 | 1 | -17/+1 |
|/ | | | | changes related to Starred nodes in the call args | ||||
* | Fix the line number of the expected message. | Claudiu Popa | 2015-10-04 | 1 | -1/+1 |
| | |||||
* | Don't emit 'assigning-non-slot' for descriptors. Closes issue #652. | Claudiu Popa | 2015-10-04 | 3 | -3/+60 |
| | |||||
* | Merged in cezarelnazli/pylint (pull request #281) | Claudiu Popa | 2015-10-04 | 3 | -0/+12 |
|\ | | | | | | | Add functional tests for PEP 448 | ||||
| * | More PEP 448 tests | Cezar | 2015-10-04 | 1 | -1/+4 |
| | | |||||
| * | Add functional tests for PEP 448 | Cezar | 2015-10-04 | 3 | -0/+9 |
| | | |||||
* | | Simplify the check. | Claudiu Popa | 2015-10-04 | 1 | -4/+5 |
| | | |||||
* | | Remove unused parameter. | Claudiu Popa | 2015-10-04 | 1 | -3/+3 |
| | | |||||
* | | Rename the name of the singleton-comparison test files. | Claudiu Popa | 2015-10-04 | 2 | -0/+0 |
| | | |||||
* | | Merged in dmand/pylint/comparison-checker (pull request #280) | Claudiu Popa | 2015-10-04 | 4 | -0/+101 |
|\ \ | |/ |/| | | | Add checker for comparisons to singleton values (True, False, None) | ||||
| * | Add initial version of comparison checkercomparison-checker | Dmitry Pribysh | 2015-10-03 | 4 | -0/+101 |
| | | | | | | | | | | It checks for expressions like 'x == True', 'x == False' and 'x == None' and suggests the correct usage ('x', 'not x', 'x is None'). | ||||
* | | Port more old tests to the new style tests and remove obsolete tests. | Claudiu Popa | 2015-10-03 | 24 | -205/+74 |
|/ |