| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
| |
positive (#8644)
* Fix a false positive for ``redundant-keyword-arg`` when a function, with a keyword-only-parameter and ``**kwargs``, is called with a positional argument and a keyword argument where the keyword argument has the same name as the positional-only-parameter.
* Add new checker ``kwarg-superseded-by-positional-arg`` which emits a warning message for the above scenario.
Closes #8558
Co-authored-by: Pierre Sassoulas <pierre.sassoulas@gmail.com>
|
|
|
|
| |
**kwargs (#8575)
|
| |
|
| |
|
| |
|
|
|
|
| |
* Fix issues with new typing Union syntax (Py310)
* Upgrade astroid to 2.14.1
|
|
|
| |
Co-authored-by: Pierre Sassoulas <pierre.sassoulas@gmail.com>
|
| |
|
|
|
|
| |
tests
|
|
|
|
| |
concatenation (#7752)
|
|
|
| |
Co-authored-by: Daniël van Noord <13665637+DanielNoord@users.noreply.github.com>
|
|
|
| |
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
|
|
|
|
| |
Co-authored-by: Mark Byrne <31762852+mbyrnepr2@users.noreply.github.com>
Co-authored-by: Pierre Sassoulas <pierre.sassoulas@gmail.com>
|
|
|
|
|
| |
assigned to a class attribute inside the class where the function is defined. (#7395)
Closes #6592
|
|
|
| |
Co-authored-by: Pierre Sassoulas <pierre.sassoulas@gmail.com>
|
|
|
|
|
|
|
| |
- Refactor Classes which inherit from `object`.
- Remove `import print_function from __future__`.
- Remove assignments to `__revision__` from the functional test module when it is never used throughout the module.
Co-authored-by: Pierre Sassoulas <pierre.sassoulas@gmail.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
* Ignore deprecated-module in access_attr_before_def_false_positive
This is because telnetlib is deprecated in Python 3.11. It's
hard to see exactly what this is testing - there's no great
explanation in-line and the test predates the first commit to
the git repo so we don't have a commit message to help.
telnetlib will be removed in 3.13, though, so at that point
we'll have to figure it out or drop the telnetlib part of the
test.
Signed-off-by: Adam Williamson <awilliam@redhat.com>
* Split asyncio.coroutine tests and set max_pyver 3.10 for them
iterable_context_py3 includes some checks that we don't emit
errors for asyncio.coroutine, but asyncio.coroutine has been
removed in Python 3.11, so we need to set a max version of
3.10 for these tests.
Signed-off-by: Adam Williamson <awilliam@redhat.com>
* Set max_pyver 3.10 for some deprecations removed in 3.11
The `binhex` module and `binascii.b2a_hqx()` function, which
were deprecated in 3.9, are removed entirely in 3.11.
Signed-off-by: Adam Williamson <awilliam@redhat.com>
* Change syntax_error test for consistent output
Python 3.11 changes the string representation of the
SyntaxError triggered by this test - it now says "expected '('"
instead of just "invalid syntax". This changes the test to use
a different error (incomplete `for` loop) which still has just
"invalid syntax" as its description in Python 3.11. This is the
same 'bad code' used in the similar `test_stdin_syntaxerror` in
the unit tests.
Signed-off-by: Adam Williamson <awilliam@redhat.com>
Co-authored-by: Daniël van Noord <13665637+DanielNoord@users.noreply.github.com>
|
|
|
|
|
| |
defining `__or__` (#7012)
Co-authored-by: Daniël van Noord <13665637+DanielNoord@users.noreply.github.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
* Use consistent location for NotFoundError
Throughout this module we're using ``astroid.NotFoundError`` as an
alias for ``astroid.exceptions.NotFoundError``, it seems best to be
consistent.
Closes #4951
Co-authored-by: Jacob Walls <jacobtylerwalls@gmail.com>
|
|
|
| |
Co-authored-by: Pierre Sassoulas <pierre.sassoulas@gmail.com>
|
|
|
| |
Co-authored-by: Pierre Sassoulas <pierre.sassoulas@gmail.com>
|
|
|
|
|
| |
Co-authored-by: Sam M W <smw@alertergroup.co.uk>
Co-authored-by: Pierre Sassoulas <pierre.sassoulas@gmail.com>
Co-authored-by: Daniël van Noord <13665637+DanielNoord@users.noreply.github.com>
|
|
|
|
|
| |
Co-authored-by: Jacob Walls <jacobtylerwalls@gmail.com>
Co-authored-by: Daniël van Noord <13665637+DanielNoord@users.noreply.github.com>
Co-authored-by: Pierre Sassoulas <pierre.sassoulas@gmail.com>
|
| |
|
|
|
|
|
|
| |
See https://github.com/PyCQA/pylint/issues/2409#issuecomment-1100952171
Closes #2409
|
| |
|
| |
|
|
|
|
| |
Instead of raising twice.
|
|
|
|
| |
Co-authored-by: Pierre Sassoulas <pierre.sassoulas@gmail.com>
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: Jacob Walls <jacobtylerwalls@gmail.com>
|
| |
|
| |
|
|
|
| |
Co-authored-by: Pierre Sassoulas <pierre.sassoulas@gmail.com>
|
| |
|
| |
|
|
|
| |
Co-authored-by: Daniël van Noord <13665637+DanielNoord@users.noreply.github.com>
|
|
|
|
|
|
|
|
|
|
|
|
| |
* Fix assigning-non-slot false positive with setattr
Previously, if a class was slotted and overrode `__setattr__`,
`assigning-non-slot` would be issued when assigning to attributes. With
`__setattr__` defined, we cannot infer if it is an error to assign to an
attribute, so we suppress the error.
Fix #3793
Co-authored-by: Pierre Sassoulas <pierre.sassoulas@gmail.com>
|
|
|
|
|
|
|
|
| |
decorator (#5412)
* Fix #5371: Correctly count arguments to static methods missing @staticmethod decorator
* Implementations of MapReduceMixin.reduce_map_data were actually not classmethods
|
| |
|
| |
|
|
|
|
|
|
| |
* Add ``confidence`` to all expected functional test outputs
* Make OutputLine accept end_line and end_column
Co-authored-by: Pierre Sassoulas <pierre.sassoulas@gmail.com>
|
|
|
|
|
|
|
| |
* Move ``misplaced-comparison-constant`` to optional extension
* Update functional tests to increase coverage
Co-authored-by: Pierre Sassoulas <pierre.sassoulas@gmail.com>
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
* Fix access to private function in inner class on protected-access bug
* Add functional test for protected-access from inner class
* Add Ikraduya to CONTRIBUTORS file
* Add if statement to avoid potential bug
* Fix ``protected-access`` for attributes and methods of nested classes
This closes #3066
Co-authored-by: ikraduya <ikraduya@gmail.com>
Co-authored-by: Pierre Sassoulas <pierre.sassoulas@gmail.com>
|
|
|
|
|
|
|
|
|
| |
* Add tests for assignment expressions in function defaults
Ref #3688
* Upgrade astroid to 2.8.4
Co-authored-by: Pierre Sassoulas <pierre.sassoulas@gmail.com>
|
| |
|
| |
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
* Fixed 'no-self-check' for async functions.
* fixed 'docparams' for async functions.
* Added 'no-self-use' to the pylint disable comment in broken test.
* Added small changelog entry to Pylint 2.11.0 milestone.
* Added test: 'no-self-use' for async method.
* Added test: revamped 'docparams' functional tests to test all missing documentation.
Co-authored-by: Marc Mueller <30130371+cdce8p@users.noreply.github.com>
Co-authored-by: Pierre Sassoulas <pierre.sassoulas@gmail.com>
|
|
|
|
|
|
| |
It's possible to use the python3 porting checker from
an older pylint version. But it's not evolving anymore
and was costing resource to maintain.
|