Commit message (Collapse) | Author | Age | Files | Lines | ||
---|---|---|---|---|---|---|
... | ||||||
| * | | check for static methods declared without a decorator | Laura M?dioni | 2015-10-29 | 6 | -8/+44 | |
| | | | | | | | | | | | | closes issue #675 | |||||
| * | | check for class methods declared without a decorator | Laura M?dioni | 2015-10-29 | 9 | -6/+76 | |
| | | | | | | | | | | | | related to the issue #675 | |||||
* | | | Add a few functional tests for enhanced unpacking checkerenhance-unpacking-checker | Dmitry Pribysh | 2015-10-29 | 2 | -13/+36 | |
| | | | ||||||
* | | | Make tuple unpacking checker emit less false negatives | Dmitry Pribysh | 2015-10-29 | 3 | -142/+151 | |
| | | | | | | | | | | | | By reusing functions from iterable checker. | |||||
* | | | Fix pylint warnings. | Claudiu Popa | 2015-10-27 | 1 | -4/+4 | |
| | | | ||||||
* | | | Merge heads. | Claudiu Popa | 2015-10-27 | 4 | -46/+164 | |
|\ \ \ | ||||||
| * | | | Add a new convention message, 'consider-using-enumerate' | Claudiu Popa | 2015-10-27 | 3 | -0/+113 | |
| | | | | | | | | | | | | | | | | | | | | | | | | The message is emitted when code that uses `range` and `len` for iterating is encountered, which can be easily simplified by using `enumerate` instead. This makes the code a bit faster and cleaner. Closes issue #684. | |||||
| * | | | Move unidiomatic-typecheck into the comparison checker from base.py, since ↵ | Claudiu Popa | 2015-10-26 | 2 | -46/+51 | |
| |/ / | | | | | | | | | | it makes more sense for it to be there. | |||||
* | | | Make iterable checker skip classes that are inferred to be abstractfix-685 | Dmitry Pribysh | 2015-10-27 | 7 | -25/+121 | |
| | | | ||||||
* | | | Skip not-an-iterable and similar checks for Base/Abstract classes | Dmitry Pribysh | 2015-10-27 | 2 | -84/+101 | |
|/ / | | | | | | | | | | | And refactor iterable checker. Fixes issue #685. | |||||
* | | Simplify visit_compare, by moving the verifications for misplaced constants ↵ | Claudiu Popa | 2015-10-26 | 1 | -4/+4 | |
| | | | | | | | | into their own if statement. | |||||
* | | Use the representation of the context when building the message for ↵ | Claudiu Popa | 2015-10-26 | 3 | -2/+4 | |
| | | | | | | | | misplaced-comparison-constant. | |||||
* | | Simplify _check_misplaced_constant. | Claudiu Popa | 2015-10-26 | 1 | -8/+7 | |
| | | ||||||
* | | Don't emit misplaced-comparison-constant for constant tests, such as true == ↵ | Claudiu Popa | 2015-10-26 | 3 | -15/+6 | |
| | | | | | | | | true, which should be handled by other checks. | |||||
* | | Use a lower message id for misplaced-comparison-constant. | Claudiu Popa | 2015-10-26 | 1 | -1/+1 | |
| | | ||||||
* | | Make check_singleton_comparison a private function. | Claudiu Popa | 2015-10-26 | 1 | -3/+3 | |
| | | ||||||
* | | Update the docstring a little bit. | Claudiu Popa | 2015-10-26 | 1 | -2/+3 | |
| | | ||||||
* | | Add an extra space between classes. | Claudiu Popa | 2015-10-26 | 1 | -0/+1 | |
| | | ||||||
* | | Make misplaced-comparison-constant a convention, rather than a warning. | Claudiu Popa | 2015-10-26 | 3 | -5/+5 | |
| | | ||||||
* | | Merge heads. | Claudiu Popa | 2015-10-26 | 13 | -32/+117 | |
|\ \ | ||||||
| * | | Explicitely check for const to const comparison | Laura M?dioni | 2015-10-26 | 3 | -16/+40 | |
| | | | | | | | | | | | | Also add a check for comparison to a method, a function or an attribute and improve the functional test accordingly. | |||||
| * | | add a new rule looking for yoda conditions | Laura M?dioni | 2015-10-15 | 13 | -32/+93 | |
| |/ | ||||||
* | | Add test for verifying that __name__ can't be accessed from an instance. | Claudiu Popa | 2015-10-26 | 2 | -0/+7 | |
| | | ||||||
* | | Remove the list parantheses for a sum call. | Claudiu Popa | 2015-10-26 | 1 | -1/+1 | |
| | | ||||||
* | | Use a generator comprehension instead of creating a list. | Claudiu Popa | 2015-10-26 | 1 | -1/+1 | |
| | | ||||||
* | | Cleanup _parallel_task by moving the creation of the config into its own ↵ | Claudiu Popa | 2015-10-26 | 1 | -3/+7 | |
| | | | | | | | | function. | |||||
* | | Add test case for issue #559. Closes issue #559. | Claudiu Popa | 2015-10-24 | 1 | -0/+7 | |
| | | ||||||
* | | Remove trailing whitespace. | Claudiu Popa | 2015-10-22 | 1 | -1/+1 | |
| | | ||||||
* | | Fix documentation build warnings and errors | Dmitry Pribysh | 2015-10-22 | 2 | -2/+2 | |
| | | | | | | | | | | | | | | | | Errors fixed: - unqouted emphasis character usage (*) - missing list index Fixes #481. | |||||
* | | Don't warn about abstract classes instantiated in their own body. Closes ↵ | Claudiu Popa | 2015-10-21 | 2 | -0/+31 | |
| | | | | | | | | issue #627. | |||||
* | | Remove trailing whitespace. | Claudiu Popa | 2015-10-20 | 1 | -1/+1 | |
| | | ||||||
* | | Obsolete options are not present by default in the generated configuration file. | Claudiu Popa | 2015-10-20 | 4 | -3/+13 | |
| | | | | | | | | Closes issue #632. | |||||
* | | non-iterator-returned can detect classes with iterator-metaclasses. | Claudiu Popa | 2015-10-20 | 3 | -18/+70 | |
| | | | | | | | | Closes issue #679. | |||||
* | | Add checker for membership rules and testsmembership-test-checker | Dmitry Pribysh | 2015-10-19 | 9 | -0/+224 | |
| | | | | | | | | | | | | | | | | Idea of checking is very simple: value used to the right of the 'in' operator should support membership test protocol (i.e. define either __contains__ or __iter__ or __getitem__). Fixes issue #589. | |||||
* | | Correct the name of the messages. | Claudiu Popa | 2015-10-18 | 1 | -2/+2 | |
| | | ||||||
* | | Disable not-iterable-context for a couple of false positives. | Claudiu Popa | 2015-10-18 | 1 | -0/+2 | |
| | | ||||||
* | | Change the numeric ids of the new checkers. | Claudiu Popa | 2015-10-18 | 2 | -2/+4 | |
| | | ||||||
* | | 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 | |
| | | |