Skip to content
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

[Backport maintenance/2.17.x] Fix unnecessary-lambda false positive for lambdas using its parameters in their body #8506

Merged

Conversation

github-actions[bot]
Copy link
Contributor

Backport b621436 from #8498.

…ers in their body (#8498)

Fixes #8496

(cherry picked from commit b621436)
@Pierre-Sassoulas Pierre-Sassoulas added this to the 2.17.2 milestone Mar 28, 2023
@Pierre-Sassoulas Pierre-Sassoulas added the False Positive 🦟 A message is emitted but nothing is wrong with the code label Mar 28, 2023
@codecov
Copy link

codecov bot commented Mar 28, 2023

Codecov Report

Merging #8506 (1bea8b9) into maintenance/2.17.x (ebf1952) will increase coverage by 0.00%.
The diff coverage is 100.00%.

Additional details and impacted files

Impacted file tree graph

@@                 Coverage Diff                 @@
##           maintenance/2.17.x    #8506   +/-   ##
===================================================
  Coverage               95.56%   95.57%           
===================================================
  Files                     178      178           
  Lines                   18780    18783    +3     
===================================================
+ Hits                    17948    17951    +3     
  Misses                    832      832           
Impacted Files Coverage Δ
pylint/checkers/base/basic_checker.py 97.88% <100.00%> (+0.01%) ⬆️

@Pierre-Sassoulas Pierre-Sassoulas merged commit d429822 into maintenance/2.17.x Mar 28, 2023
@Pierre-Sassoulas Pierre-Sassoulas deleted the backport-8498-to-maintenance/2.17.x branch March 28, 2023 20:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
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.

2 participants