Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Announce the move to github in the readme file.HEADmaster | Claudiu Popa | 2015-12-10 | 1 | -57/+2 |
| | |||||
* | Change the attribute check for using hasattr instead of looking explicitly ↵ | Claudiu Popa | 2015-12-08 | 1 | -1/+1 |
| | | | | into __dict__. | ||||
* | Give a nice error message when Graphviz is not installed. Closes issue #168. | Emile Anclin | 2015-12-08 | 2 | -0/+20 |
| | |||||
* | Use the new .ctx attribute for subscripts for detecting in which context the ↵ | Claudiu Popa | 2015-12-08 | 1 | -25/+9 |
| | | | | node was used. Closes issue #705. | ||||
* | Merged in jakirkham/pylint/issue_538_pt2 (pull request #315) | Claudiu Popa | 2015-12-07 | 17 | -70/+224 |
|\ | | | | | | | Add another check of the current working directory for `.pylintrc`. | ||||
| * | Allow statements in if or try blocks containing imports. | Laura M?dioni | 2015-12-02 | 4 | -8/+39 |
| | | | | | | | | Closes issue #714 | ||||
| * | Move the construction of generated_members into open. | Claudiu Popa | 2015-12-07 | 1 | -10/+10 |
| | | |||||
| * | Added a new error, 'relative-beyond-top-level'. | Claudiu Popa | 2015-12-06 | 5 | -10/+57 |
| | | | | | | | | | | | | | | This is emitted when a relative import was attempted beyond the top level package. For instance, if a package has X levels, trying to climb X + n levels with a relative import, as in `from ..stuff import Stuff`, will result in an error. Closes issue #588. | ||||
| * | Make pylint work with new astroid exceptions, AstroidImportError and ↵ | Claudiu Popa | 2015-12-06 | 5 | -23/+19 |
| | | | | | | | | AstroidSyntaxError. | ||||
| * | Don't emit super-on-old-class on classes with unknown bases. | Claudiu Popa | 2015-12-06 | 4 | -23/+23 |
| | | | | | | | | | | | | The change also removes the confidence handling for super-on-old-class, which isn't used enough to merit its existence. Closes issue #721. | ||||
| * | Accept only functions and methods for the deprecated-method checker. | Claudiu Popa | 2015-12-03 | 3 | -3/+83 |
| | | | | | | | | | | This prevents a crash which can occur when an object doesn't have .qname() method after the inference. | ||||
* | | pylint/config.py: Add another check of the current working directory for ↵issue_538_pt2 | John Kirkham | 2015-12-02 | 1 | -0/+2 |
|/ | | | | `.pylintrc`. | ||||
* | Disable persistence for test_self. | Claudiu Popa | 2015-12-02 | 1 | -0/+1 |
| | |||||
* | Try to register the else-if checker explicitly instead on relying on path ↵ | Claudiu Popa | 2015-12-02 | 1 | -4/+2 |
| | | | | assumptions. | ||||
* | Remove import which was imported twice. | Claudiu Popa | 2015-12-01 | 1 | -1/+1 |
| | |||||
* | Don't emit import-self and cyclic-import for relative imports of modules ↵ | Claudiu Popa | 2015-12-01 | 6 | -4/+40 |
| | | | | | | | | | | | | | | | | | with the same name as the package itself. The problem was partially the fault of astroid.modutils.get_module_part, in combination with a given context file. The function returned 'dummy' as the module part for the string `dummy.dummy.Dummy`, which is in fact true, since the first dummy is the package and the second dummy is the module from where Dummy gets loaded. But get_module_part has no way to know this semantic inference, that the second dummy is a relative import inside the first one. As such, it's better to just skip the check if the condition of being relative inside a __init__.py file is found, since there's no way to load itself in that case. Closes issues #708 and #706. | ||||
* | Refactor things through the imports checker | Claudiu Popa | 2015-12-01 | 4 | -67/+92 |
| | | | | | | | | This patch transforms some public functions / methods to private and moves some blocks of code into their own functions. Through the latter, a couple of new messages are now emitted even though the module couldn't be imported, such as reimported, which doesn't make sense to not emit in this case. | ||||
* | Add else-if-used rule functional test | Laura M?dioni | 2015-11-30 | 2 | -0/+75 |
| | |||||
* | fix elif-used rule | Laura M?dioni | 2015-11-30 | 1 | -18/+4 |
| | | | | The bug was introduced when porting the rule from pylint core to an extension | ||||
* | Add changelog entry for 0c2ba76 | Claudiu Popa | 2015-11-30 | 1 | -0/+9 |
| | |||||
* | Fix a crash which occurred when old visit methods are encountered | Claudiu Popa | 2015-11-30 | 3 | -9/+29 |
| | | | | | | | | | | in plugin modules. If a plugin uses an old visit method (visit_class for instance), this can lead to a crash in pylint's base checkers, because the logic in the PylintASTWalker assumes that all checkers have a visit_class / leave_class method. The patch fixes this by looking for both names. Closes issue #711. | ||||
* | Don't emit unsubscriptable-object if the node is found inside an abstract ↵ | Claudiu Popa | 2015-11-30 | 2 | -0/+28 |
| | | | | class. Closes #685. | ||||
* | Add wrong-import-position to check_messages's decorator arguments for ↵ | Claudiu Popa | 2015-11-30 | 4 | -2/+38 |
| | | | | | | | | | | ImportChecker.leave_module This fixes an esoteric bug which occurs when ungrouped-imports and wrong-import-order are disabled and pylint is executed on multiple files. What happens is that without wrong-import-position in check_messages, leave_module will never be called, which means that the first non-import node from other files might leak into the current file, leading to wrong-import-position being emitted by pylint. | ||||
* | Don't use .iteritems, it doesn't exist on Python 3. Change some variable ↵ | Claudiu Popa | 2015-11-29 | 1 | -11/+8 |
| | | | | names and the name of the builtins module. | ||||
* | pylint extension to check for use of "else if" instead of "elsif" | Laura M?dioni | 2015-11-27 | 1 | -0/+76 |
| | | | | related to issue #673 | ||||
* | Add some more comments about the limitations of this rule | Laura M?dioni | 2015-11-24 | 2 | -3/+10 |
| | | | | related to issue #674 | ||||
* | Drop print | Laura M?dioni | 2015-11-24 | 1 | -2/+0 |
| | | | | related to issue #674 | ||||
* | Use a stack of dictionaries instead of 3 dicts | Laura M?dioni | 2015-11-24 | 3 | -30/+25 |
| | | | | | | | - This way it works with embedded classes and functions - Update the tests accordingly related to issue #674 | ||||
* | Remove test on str.split() since there is no support for inference on split ↵ | Laura M?dioni | 2015-11-24 | 2 | -11/+5 |
| | | | | | | for now related to issue #674 | ||||
* | check if the type of a variable is redefined (at a function, class or module ↵ | Laura M?dioni | 2015-10-30 | 8 | -5/+149 |
| | | | | | | scope) related to issue #674 | ||||
* | Added a new warning, 'unsupported-delete-operation' | Claudiu Popa | 2015-11-26 | 5 | -21/+154 |
| | | | | | It is emitted when item deletion is tried on an object which doesn't have this ability. Closes issue #592. | ||||
* | Added a new warning, 'unsupported-assignment-operation' | Claudiu Popa | 2015-11-26 | 5 | -9/+162 |
| | | | | | This is emitted when item assignment is tried on an object which doesn't have this ability. Closes issue #591. | ||||
* | Simplify the protocol methods, by using a common function. | Claudiu Popa | 2015-11-26 | 1 | -41/+12 |
| | |||||
* | Ignore multiple comparisons in unneeded-not rule | Laura M?dioni | 2015-11-25 | 2 | -0/+7 |
| | | | | related to issue #703 | ||||
* | Make some imports external instead of considering them local | Claudiu Popa | 2015-11-25 | 1 | -3/+3 |
| | | | | | | | | | The assumption that a module which can't be imported is local rather than external isn't always correct, since the implementation of the astroid's import system should always find a module which is local in the current package, while it isn't necessary true that it can find always external modules, which just might be missing. | ||||
* | Change the variable name to something more suggestive | Claudiu Popa | 2015-11-25 | 1 | -6/+7 |
| | |||||
* | Fix some docstrings and the Python 3 tests | Claudiu Popa | 2015-11-25 | 5 | -8/+10 |
| | |||||
* | Add ChangeLog entry for issue #692. | Claudiu Popa | 2015-11-25 | 1 | -1/+11 |
| | |||||
* | Fix new import related errors in pylint's codebase. | Claudiu Popa | 2015-11-25 | 20 | -45/+51 |
| | |||||
* | Improve a bit wrong-import-position code and add comments | Laura M?dioni | 2015-11-24 | 1 | -7/+11 |
| | | | | related to issue #692 | ||||
* | Simplify a bit the algorithm checking for ungrouped imports | Laura M?dioni | 2015-11-24 | 1 | -12/+8 |
| | | | | related to issue #692 | ||||
* | Remove a chunk of code that was never executed | Laura M?dioni | 2015-11-23 | 1 | -3/+0 |
| | | | | related to issue #692 | ||||
* | Factorize code in _record_import method | Laura M?dioni | 2015-11-23 | 1 | -9/+13 |
| | | | | related to issue #692 | ||||
* | slightly change wrong-import-position message and add forgotten functional test | Laura M?dioni | 2015-11-12 | 3 | -2/+26 |
| | | | | related to issue #692 | ||||
* | Check imports are located at the top of the module (right after docstring ↵ | Laura M?dioni | 2015-11-10 | 42 | -63/+104 |
| | | | | | | and comments) related to issue #692 | ||||
* | Check imports are ordered (standard, 3rd party, local) and grouped by package | Laura M?dioni | 2015-11-09 | 14 | -10/+132 |
| | | | | related to issue #692 | ||||
* | Add ChangeLog entry for issue #670. | Claudiu Popa | 2015-11-25 | 1 | -0/+4 |
| | |||||
* | Fix instances of unneeded-not in pylint's codebase. | Claudiu Popa | 2015-11-25 | 8 | -9/+8 |
| | |||||
* | Add is to the checked operators | Laura M?dioni | 2015-11-24 | 3 | -1/+4 |
| | |||||
* | Fix crash with 'in' operator on unneeded-not | Laura M?dioni | 2015-10-29 | 3 | -2/+9 |
| |