| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
|
|
|
|
| |
* [overgeneral-exceptions] Only handle qualified names
* fix changelog
|
| |
|
|
|
| |
Co-authored-by: Daniël van Noord <13665637+DanielNoord@users.noreply.github.com>
|
|
|
| |
125 is a good start. The check was activated in pylint with value = 100, but flake8 is less lenient than pylint and does not make any exceptions (for docstrings, strings and comments in particular).
|
|
|
|
|
| |
except statement if exception are in different namespace (#8052)
Closes #8051
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
* Update tests
* Use qualified name when checking for overgeneral exceptions
* WIP: Add deprecation warning
* Add changelog fragment
* Use qualified name in test case
* spell check fix
* Update changelog fragment with suggested fixes
Co-authored-by: Daniël van Noord <13665637+DanielNoord@users.noreply.github.com>
* Move OVERGENERAL_EXCEPTIONS directly to the default value in dict
* Mark as TODO for pylint 3.0
* Properly warn on each occurrence of name without dots
* Update the warning per the review
* Rephrase the warning to mention pylint 3.0
* Remove unnecessary nesting of the if condition
* Quote the exception name in deprecation warning
* Use config value for overgeneral exceptions in broad-exception-raised
* Infer qualified name of the exception in broad-exception-raised
* e.g. -> maybe?
Co-authored-by: Pierre Sassoulas <pierre.sassoulas@gmail.com>
* Suppress missing class docstrings
* Add few more tests for broad-exception-raised
Co-authored-by: Pierre Sassoulas <pierre.sassoulas@gmail.com>
* Fix unexpected missing-raise-from
* Revert "Fix unexpected missing-raise-from"
This reverts commit d796e72035b7f7578b9e6bb1e45a30935e80b009.
* Revert "Add few more tests for broad-exception-raised"
This reverts commit e5a193ee136f8566d43450fbb9fbf28cc717d307.
* Change confidence of broad-exception-raised from HIGH to INFERENCE
* Only trigger broad-exception-raised for raise with new exc instance
* Update overgeneral-exceptions definition in example pylintrc file
* Update pylint/checkers/exceptions.py
Co-authored-by: Daniël van Noord <13665637+DanielNoord@users.noreply.github.com>
Co-authored-by: Pierre Sassoulas <pierre.sassoulas@gmail.com>
|
| |
|
|
|
|
| |
Co-authored-by: Pierre Sassoulas <pierre.sassoulas@gmail.com>
|
|
|
|
|
|
| |
Fixes #3694
Co-authored-by: Victor Taïx <victor.taix@grenoble-inp.org>
Co-authored-by: Pierre Sassoulas <pierre.sassoulas@gmail.com>
|
| |
|
| |
|
|
|
| |
Co-authored-by: Marc Mueller <30130371+cdce8p@users.noreply.github.com>
|
|
|
|
|
|
|
|
|
| |
* [raise-missing-from] Clearer message and example in the documentation
Co-authored-by: cool-RR <ram@rachum.com>
Co-authored-by: Daniël van Noord <13665637+DanielNoord@users.noreply.github.com>
Refs #5953
Closes #3707
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Text was:
```
--overgeneral-exceptions <comma-separated class names>
Exceptions that will emit a warning when being caught. Defaults to "BaseException, Exception". (default: ('BaseException', 'Exception'))
```
Now:
```
--overgeneral-exceptions <comma-separated class names>
Exceptions that will emit a warning when being caught. (default: ('BaseException', 'Exception'))
```
Co-authored-by: Daniël van Noord <13665637+DanielNoord@users.noreply.github.com>
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
|
|
| |
Co-authored-by: Pierre Sassoulas <pierre.sassoulas@gmail.com>
|
|
|
| |
Co-authored-by: Pierre Sassoulas <pierre.sassoulas@gmail.com>
|
|
|
|
|
| |
Fix the existing file so they have a notice.
No header for setup.py or examples or doc
|
|
|
|
|
|
| |
git is the source of truth for the copyright, copyrite (the tool)
was taking exponentially longer with each release, and it's
polluting the code with sometime as much as 50 lines of names.
|
| |
|
| |
|
| |
|
|
|
|
|
| |
Remove verbose docstring in code, keep them in example and doc
Co-authored-by: Marc Mueller <30130371+cdce8p@users.noreply.github.com>
|
|
|
|
| |
Co-authored-by: Daniël van Noord <13665637+DanielNoord@users.noreply.github.com>
Co-authored-by: Jacob Walls <jacobtylerwalls@gmail.com>
|
|
|
|
| |
Closes #5250
|
|
|
|
|
|
|
|
|
|
|
| |
* Fix typo in pylintrc for useless-suppression
* Fix CI for information messages
* Remove useless disables
* Update CI and docs for 3.8+
Co-authored-by: Pierre Sassoulas <pierre.sassoulas@gmail.com>
|
|
|
|
|
|
|
| |
* Enable for_any_all check, reword a few docstrings, shorten some loop variable
names, and add typing on modified functions.
Co-authored-by: Pierre Sassoulas <pierre.sassoulas@gmail.com>
Co-authored-by: Daniël van Noord <13665637+DanielNoord@users.noreply.github.com>
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
* Add type annotations to ``visit`` & ``leave`` calls
This adds typing to most calls that visit nodes. All other changes are
due to mypy errors resulting from introduction of typing.
* Fix outstanding mypy issues
This removes some of the `type: ignore` comments in favour of
solving the mypy issues these comments were surpressing.
* Fix remaining references to node_classes
Except for two references to node_classes in the changelog this should be the last of them
Co-authored-by: Marc Mueller <30130371+cdce8p@users.noreply.github.com>
Co-authored-by: Pierre Sassoulas <pierre.sassoulas@gmail.com>
|
|
|
|
|
|
| |
Apparently this use case was supposed to be handled by the py3K
checker but as it was disabled all the time the issue has'nt
been handled at all for a long time.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
* Add ``consider-using-f-string`` checker
This adds a checker for normal strings which are formatted
with ``.format()`` or '%'.
The message is a convention to nudge users towards using f-strings.
This closes #3592
* Update pylint code to use f-strings
After adding `consider-using-f-strings` the codebase showed numerous
cases of formatting which could be f-strings.
This commit changes most of these to become f-strings, or adds ignores.
* Apply suggestions from code review
Co-authored-by: Pierre Sassoulas <pierre.sassoulas@gmail.com>
|
|
|
|
| |
* Use alias for astroid nodes
* Resolve name conflicts
|
| |
|
|
|
|
| |
* Fix link in license header
* Update link to astroid bump_changelog
|
| |
|
|
|
|
|
| |
* Change msg name to include dataclass
* Update pylint ignores
* Update tests
|
|
|
|
| |
* Add new checker consider-using-namedtuple
* Add disable for existing code base
|
| |
|
| |
|
| |
|
|
|
|
| |
Remove unused imports for astroid
|
| |
|
| |
|