only one error of each type per line about prospector HOT 2 CLOSED

There's a special case for pylint to collapse multiple unused imports into one message. This came about because of from foo import * could cause 100 unused import warnings. However, from your example, I agree there should be one per unused import when multiple imports are explicitly specified. I'll change the logic to only collapse if a * import is used.

from prospector.

Wouldn't it be best to throw the warning that import * is wrong and silence further errors from the import statement instead of actually checking for all the unused imported results of an import * statement? It's pretty much universally accepted that import * is bad code! is it actually necessary to check what it imports and use those results? It should either be replaced with explicit imports or a library/module import and use of the module/library prefix. 

Sent from Molto for iPad

From: carlio
Sent: Tuesday, July 08, 2014 01:08 am
To: landscapeio/prospector
Cc: Samuel Bishop
Subject: Re: [prospector] only one error of each type per line (#30)

There's a special case for pylint to collapse multiple unused imports into one message. This came about because of from foo import * could cause 100 unused import warnings. However, from your example, I agree there should be one per unused import when multiple imports are explicitly specified. I'll change the logic to only collapse if a * import is used.

—
Reply to this email directly or view it on GitHub.

from prospector.