Skip to content

Conversation

mbyrnepr2
Copy link
Member

@mbyrnepr2 mbyrnepr2 commented Apr 24, 2023

Type of Changes

Type
βœ“ πŸ› Bug fix
✨ New feature
πŸ”¨ Refactoring
πŸ“œ Docs

Description

Fix a false positive for bad-dunder-name when there is a user-defined __index__ method.

Closes #8613

@mbyrnepr2 mbyrnepr2 added this to the 2.17.4 milestone Apr 24, 2023
@mbyrnepr2 mbyrnepr2 added the False Positive 🦟 A message is emitted but nothing is wrong with the code label Apr 24, 2023
@github-actions

This comment has been minimized.

@mbyrnepr2 mbyrnepr2 force-pushed the 8613_bad_dunder_name branch from 1329e35 to ea61dc9 Compare April 24, 2023 17:27
@mbyrnepr2 mbyrnepr2 force-pushed the 8613_bad_dunder_name branch from ea61dc9 to 94cd2e3 Compare April 24, 2023 17:50
@codecov
Copy link

codecov bot commented Apr 24, 2023

Codecov Report

Merging #8619 (94cd2e3) into main (a83137d) will not change coverage.
The diff coverage is n/a.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #8619   +/-   ##
=======================================
  Coverage   95.91%   95.91%           
=======================================
  Files         174      174           
  Lines       18416    18416           
=======================================
  Hits        17664    17664           
  Misses        752      752           
Impacted Files Coverage Ξ”
pylint/constants.py 100.00% <ΓΈ> (ΓΈ)

Copy link
Member

@Pierre-Sassoulas Pierre-Sassoulas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, but let's wait for 2.17.3 release now (#8620)

@Pierre-Sassoulas Pierre-Sassoulas merged commit f223c6d into pylint-dev:main Apr 24, 2023
@github-actions
Copy link
Contributor

πŸ€– According to the primer, this change has no effect on the checked open source code. πŸ€–πŸŽ‰

This comment was generated for commit 94cd2e3

@mbyrnepr2 mbyrnepr2 deleted the 8613_bad_dunder_name branch April 24, 2023 18:29
github-actions bot pushed a commit that referenced this pull request Apr 24, 2023
…ined ``__index__`` method. (#8619)

Closes #8613

(cherry picked from commit f223c6d)
Pierre-Sassoulas pushed a commit that referenced this pull request Apr 24, 2023
…ined ``__index__`` method. (#8619) (#8622)

Closes #8613

(cherry picked from commit f223c6d)

Co-authored-by: Mark Byrne <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Backported False Positive 🦟 A message is emitted but nothing is wrong with the code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Bad dunder name is flagging __index__

2 participants