anchore / anchore-engine

A service that analyzes docker images and scans for vulnerabilities
Apache License 2.0
1.58k stars 271 forks source link

Remove all unrecognized pylint options and remove consider-using-generator specifically which was causing the errors #1382

Closed dakaneye closed 2 years ago

dakaneye commented 2 years ago

Signed-off-by: Samuel Dacanay sam.dacanay@anchore.com

What this PR does / why we need it: lint job was failing in CI

Which issue this PR fixes (optional, in fixes #<issue number>)(, fixes #<issue_number, ...) format, will close the issue when PR is merged: fixes #: lint job

Special notes:

From vijay's old comment:

we were specifically enabling checks because the output is too large to fix in one go. As cleanups are done, more checks can be enabled It looks like a bunch of the specific checks we added are no longer recognized, so we can probably remove them.

The specific lint error had to do w/ using generators, which seemed optional, so I removed it.