-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merge maintenance/2.15.x in main following 2.15.6 release #7799
Merge maintenance/2.15.x in main following 2.15.6 release #7799
Conversation
Co-authored-by: Pierre Sassoulas <[email protected]>
Co-authored-by: Pierre Sassoulas <[email protected]>
* short-circuit if next method doesnt have args * check for builtins.next qname * add inference confidence level
* Remove restore keys * Log pylint + astroid versions * Reset cache versions * Add check-latest to setup-python * Use pyproject.toml for hash * Update comment-hider version comment * Pin additional actions
* Use release version for astroid * Use better cache key * Mirror create environment * Update comments
* Fix a crash when a child class with an ``__init__`` method inherits from a parent class with an ``__init__`` class attribute. * `continue` if not a method. * Update pylint/checkers/classes/class_checker.py * Rename fragment Closes pylint-dev#7742 Co-authored-by: Pierre Sassoulas <[email protected]> Co-authored-by: Daniël van Noord <[email protected]>
…dev#7685) * do not report unnecessary list index lookup if start arg is passed * account for calling start with 0 or negative num Co-authored-by: Pierre Sassoulas <[email protected]>
…`. (pylint-dev#7757) Closes pylint-dev#7501 Co-authored-by: Pierre Sassoulas <[email protected]>
Co-authored-by: Pierre Sassoulas <[email protected]>
* Bump pylint to 2.15.6, update changelog Co-authored-by: Jacob Walls <[email protected]>
42baa8c
to
2f94b72
Compare
Pull Request Test Coverage Report for Build 3502789132
💛 - Coveralls |
Messages sent to reporter are now copied so a reporter cannot modify the message sent to other reporters. | ||
|
||
Closes #7214 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This fragment should probably be removed as it's included in the changelog for 2.15.6
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it's voluntary to include the released notes from patches in minor's whatsnew. It's also convenient as when you merge back the removal need to be done manually again.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I can't find the discussion about it :( Here we go : #7426 (comment)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The only reason we have a diff on this fragment versus the others is because I suggested a grammar fix on this fragment that Pierre committed at release time.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The only reason we have a diff on this fragment versus the others is because I suggested a grammar fix on this fragment that Pierre committed at release time.
👍🏻
I think it's voluntary to include the released notes from patches in minor's whatsnew. It's also convenient as when you merge back the removal need to be done manually again.
I was thinking about it the wrong way. We still "need" them for main
as they technically aren't released yet, just backported.
No description provided.