Add new checker (unnecessary-comprehension) (close #2905) #2999

pheanex · 2019-07-10T17:47:53Z

Steps

Add yourself to CONTRIBUTORS if you are a new contributor.
Add a ChangeLog entry describing what your PR does.
If it's a new feature or an important bug fix, add a What's New entry in doc/whatsnew/<current release.rst>.
Write a good description on what the PR does.

Add a new checker "unnecessary-comprehension" (See #2905)

	Type
✓	✨ New feature

Closes #2905

I mean its ugly now, but if you want it like that: Here you go

coveralls · 2019-07-12T17:30:23Z

Coverage increased (+0.01%) to 89.889% when pulling c354006 on pheanex:feature-2905 into f5cd858 on PyCQA:master.

PCManticore · 2019-07-16T05:55:22Z

This was a great PR to review @pheanex ! Thank you.

PHeanEX added 2 commits July 10, 2019 19:41

Add new checker (unnecessary-comprehension) (close pylint-dev#2905)

94ce45e

blacken the code

c354006

I mean its ugly now, but if you want it like that: Here you go

PCManticore merged commit 0a6b0c5 into pylint-dev:master Jul 16, 2019

pheanex deleted the feature-2905 branch July 16, 2019 15:01

pheanex restored the feature-2905 branch July 16, 2019 15:02

pheanex deleted the feature-2905 branch July 16, 2019 15:02

pheanex restored the feature-2905 branch July 16, 2019 15:02

jacobtylerwalls mentioned this pull request Oct 21, 2023

fix unnecessary-comprehension provides incorrect suggestion #9172

Merged