Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Control the maximum number of inference values from pylint with ↵ | Claudiu Popa | 2018-07-15 | 1 | -0/+7 |
| | | | | --limit-inference-results | ||||
* | Fix incorrect hanging indent detection for with statements | Bryce Guinta | 2018-07-15 | 2 | -3/+14 |
| | | | | | | | | Add with statement to recognized token keywords for format checker. Also recognize with statement as the start of a block to allow deeper indentation for hanging continuation Close #461 | ||||
* | Add docstring to _hanging_indent_after_bracket | Bryce Guinta | 2018-07-15 | 1 | -1/+10 |
| | |||||
* | Rename pos to position to keep naming consistent | Bryce Guinta | 2018-07-15 | 1 | -6/+6 |
| | |||||
* | Fix lint error | Bryce Guinta | 2018-07-13 | 1 | -0/+2 |
| | | | | astroid incorrectly inferring NoneType because of name reassignment | ||||
* | Disable false-positive test message from changes in astroid | Bryce Guinta | 2018-07-13 | 2 | -1/+2 |
| | |||||
* | Do not try to lower checker name if not set | Bryce Guinta | 2018-07-13 | 1 | -1/+2 |
| | |||||
* | Return early when possible in typecheck call visitor | Nick Drozd | 2018-07-13 | 1 | -19/+21 |
| | | | | This avoids doing unnecessary work in many cases. | ||||
* | Avoid ancestors call in ABC check if possible | Nick Drozd | 2018-07-13 | 1 | -3/+10 |
| | |||||
* | Return early to avoid some nodes_of_class calls | Nick Drozd | 2018-07-13 | 1 | -0/+7 |
| | |||||
* | Fix false positive unused-variable in lambda default arguments (#2276) | Bryce Guinta | 2018-07-12 | 4 | -7/+7 |
| | | | | | | | | | | The NameConsumer to_consume frame needed to be backed out by one so that for lambda bound default argument variables (closure) could consume the name in a higher scope (just like function default arguments) Close #1921 Close #1552 Close #1099 Close #210 | ||||
* | Fix false-postive undefined-variable in nested lambda. (#2274) | Bryce Guinta | 2018-07-12 | 2 | -1/+13 |
| | | | | | Remove unnecessary break in checker utils for lambdas causing parent lambdas to be ignored. Close #760 | ||||
* | Refactor duplicate code in is_defined_before | Bryce Guinta | 2018-07-11 | 1 | -5/+1 |
| | | | | isinstance can take multiple tuples | ||||
* | Fix lint error | Claudiu Popa | 2018-07-11 | 1 | -1/+1 |
| | |||||
* | frozenset is an iterating context for dict.values() and friends | Claudiu Popa | 2018-07-10 | 1 | -1/+1 |
| | |||||
* | `trailing-comma-tuple` can be emitted for `return` statements as well. | Claudiu Popa | 2018-07-10 | 3 | -1/+11 |
| | | | | Close #2269 | ||||
* | Filter with should_analyze_file in parallel mode (#2264) | Randall Leeds | 2018-07-10 | 2 | -14/+21 |
| | | | | | | | | | | Each ChildLinter receives a file path and instantiates a linter to check that file. As such, all files are arguments to child linters in parallel mode. Therefore, the check for should_analyze_file must happen in the parent linter, where knowledge of the original arguments is available. Expand the custom should_analyze_file test to exercise parallel mode. Close #1885 | ||||
* | Infer the value of the truth_value before looking for consider-using-ternary ↵ | Claudiu Popa | 2018-07-08 | 3 | -4/+18 |
| | | | | | | | | | | and simplifiable-boolean-expression The reason for that is that a Name node, for instance, will not have a bool_value() implementation, thus it will default to returning Uninferable. In order to avoid that, just infer the object before verifying anything about it. Related to #2058 | ||||
* | Do not emit consider-using-ternary when all the elements involved are a ↵ | Claudiu Popa | 2018-07-08 | 2 | -0/+6 |
| | | | | Compare node (might not be simplifiable to a ternary). | ||||
* | Remove test that spends more than 60 seconds to run. | Claudiu Popa | 2018-07-06 | 2 | -12613/+0 |
| | |||||
* | Fix test errors from fixing astroid.as_string() | Bryce Guinta | 2018-07-06 | 6 | -12/+12 |
| | |||||
* | Check Slice is type index (fix #2252) | Konstantin Manna | 2018-07-05 | 2 | -0/+4 |
| | |||||
* | Prepare a new dev release | Claudiu Popa | 2018-07-05 | 1 | -1/+1 |
| | |||||
* | Emit consider-using-get only if the target is a Name, as this is the pattern ↵ | Claudiu Popa | 2018-07-05 | 2 | -0/+8 |
| | | | | | | we try to detect Close #2254 | ||||
* | Fix lint | Claudiu Popa | 2018-07-04 | 1 | -1/+3 |
| | |||||
* | Only accept simple for-loop bodies when looking for consider-using-join | Claudiu Popa | 2018-07-04 | 2 | -5/+17 |
| | | | | | | Also verify that the assignment targets are actually AssignName Close #2250 | ||||
* | Fix wrong continued indentation (bad-continuation) (#2244) | Steven M. Vascellaro | 2018-07-03 | 1 | -6/+2 |
| | | | Fixes a pylint warning for wrong continued indentation (bad-continuation) | ||||
* | Add docstrings for naming style classes (#2245) | Steven M. Vascellaro | 2018-07-03 | 1 | -0/+4 |
| | | | Add docstrings for SnakeCaseStyle, CamelCaseStyle, PascalCaseStyle, and UpperCaseStyle | ||||
* | `ImportFrom` nodes correctly use the full name for the import sorting checks. | Claudiu Popa | 2018-07-03 | 3 | -6/+10 |
| | | | | Close #2181 | ||||
* | Expand ignored-argument-names include starred arguments and keyword ↵ | Bryce Guinta | 2018-07-03 | 2 | -2/+25 |
| | | | | arguments (#2237) | ||||
* | Acknowledge that ellipses are allowed in typing annotations (#2236) | Bryce Guinta | 2018-07-02 | 2 | -1/+8 |
| | | | Prevents false-positive bad-whitespace message | ||||
* | Fix of false positive ``ìnconsistent-return-statements`` (#2220) | hippo91 | 2018-06-28 | 3 | -17/+27 |
| | | | | | Correcting the way if statements are determined as return ended or not. Close #1782 | ||||
* | fix logic error in comment (#2217) | Konstantin | 2018-06-28 | 1 | -1/+1 |
| | |||||
* | Merge branch 'master' into fix_2177 | Sushobhit | 2018-06-22 | 20 | -84/+153 |
|\ | |||||
| * | Use as_string() to get the representation of an annotation, not name, which ↵ | Claudiu Popa | 2018-06-21 | 2 | -2/+10 |
| | | | | | | | | can be missing. Close #2210 | ||||
| * | Add a check `consider-using-set-comprehension` which is emitted if for set ↵ | Claudiu Popa | 2018-06-20 | 3 | -7/+26 |
| | | | | | | | | | | | | initialization the old style with list comprehensions is used. | ||||
| * | Use the latest astroid | Claudiu Popa | 2018-06-20 | 1 | -1/+1 |
| | | |||||
| * | [].extend and similar builtin operations don't emit `dict-*-not-iterating` ↵ | Claudiu Popa | 2018-06-20 | 2 | -0/+16 |
| | | | | | | | | | | | | with the Python 3 porting checker Close #2187 | ||||
| * | Install the latest dev astroid to get typed_ast | Claudiu Popa | 2018-06-20 | 1 | -2/+2 |
| | | |||||
| * | Add new checker 'consider-using-dict-comprehension' (#2204) | Konstantin | 2018-06-20 | 3 | -2/+28 |
| | | | | | | | | | | A new check is emitted for using old style dict initializations with list comprehensions instead of dict-comprehensions, which are faster and easier to read. | ||||
| * | standardize help strings a bit (#2207) | Mike Frysinger | 2018-06-20 | 10 | -79/+79 |
| | | | | | | | | | | | | | | A number of help strings are inconsistent in trailing periods, or spaces after them in the middle of text, or add a large amount of whitespace by using \ to continue the string. Standardize a lot of these so all help messages end in a period and there is normal whitespace in the middle. | ||||
* | | adapt existing test cases for useless-object-inheritance. | ssolanki | 2018-06-21 | 144 | -144/+154 |
| | | | | | | | | | | | | | | | | | | | | adapt existing test/input test cases to new check. add some more positive test case for useless-object-inheritance msg code was in conflict with existing msg. adapt existing test cases for new checker. | ||||
* | | fix lint issue w.r.t useless-object-inheritance. | ssolanki | 2018-06-20 | 6 | -9/+9 |
| | | |||||
* | | Add new checker useless-object-inheritance. | ssolanki | 2018-06-20 | 18 | -27/+59 |
|/ | | | | Close #2177 | ||||
* | Disable an insane linting error | Claudiu Popa | 2018-06-18 | 1 | -0/+1 |
| | |||||
* | `logging-not-lazy` is emitted whenever pylint infers that a string is built ↵ | Claudiu Popa | 2018-06-18 | 3 | -6/+14 |
| | | | | | | with addition Close #2193 | ||||
* | Add checker for simplifying chained comparison. | Sushobhit | 2018-06-18 | 4 | -2/+134 |
| | | | | | | | Add new checker "chained-comparison" which is used when pylint can detect that multiple comparisons can be squashed together into one. Close #2032 | ||||
* | Cut astroid.Repr | Nick Drozd | 2018-06-18 | 1 | -1/+1 |
| | | | | This type of node doesn't exist in Python 3. | ||||
* | Adapt the test to pass with the recent astroid changes | Claudiu Popa | 2018-06-15 | 2 | -4/+10 |
| | |||||
* | incorporate review comments. | ssolanki | 2018-06-15 | 2 | -26/+16 |
| |