Skip to content
This repository has been archived by the owner on Oct 19, 2021. It is now read-only.

Feature: unused imports #235

Closed
Zearin opened this issue Feb 24, 2014 · 3 comments
Closed

Feature: unused imports #235

Zearin opened this issue Feb 24, 2014 · 3 comments

Comments

@Zearin
Copy link

Zearin commented Feb 24, 2014

I’d like it if coffeelint reported any time that a file requires a module that is never used in the same file.

@AsaAyers
Copy link
Collaborator

Is there a difference between what you're requesting and detecting unused variables?

path = require 'path'

Once #20 gets completed you should get a warning if the variable path is unused.

@Zearin
Copy link
Author

Zearin commented Feb 24, 2014

Is there a difference between what you're requesting and detecting unused variables?

Oh! Come to think of it, I suppose not. …Well…there might be?

It’s effectively the same thing. But I think it might be worth highlighting this as a more severe case of unused variable. I suspect that an unused import has the potential to be more wasteful than the run-of-the-mill unused variable.

I could be wrong, though.

@AsaAyers
Copy link
Collaborator

You're right about them being more severe, but I think it would have to be detected the same way, so I'm going to close this and consider it a duplicate.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants