Skip to content

Delete an unused import#1744

Merged
mergify[bot] merged 2 commits into
haskell:masterfrom
ndmitchell:remove-unused-import
Apr 17, 2021
Merged

Delete an unused import#1744
mergify[bot] merged 2 commits into
haskell:masterfrom
ndmitchell:remove-unused-import

Conversation

@ndmitchell

Copy link
Copy Markdown
Collaborator

Not sure why we don't have a warning check for unused imports? But this one seems to have slipped through.

@pepeiborra

Copy link
Copy Markdown
Collaborator

This may not be the only redundant import.

@ndmitchell

Copy link
Copy Markdown
Collaborator Author

It was the only one I saw in Ghcide, as the scrawl was flying past. There were a few in other plugins. I'm not a fan of -Wall, but I do enable this warning in all my projects, since it can help with build times.

@ndmitchell ndmitchell added the merge me Label to trigger pull request merge label Apr 17, 2021
@mergify
mergify Bot merged commit 3152c35 into haskell:master Apr 17, 2021
@ndmitchell
ndmitchell deleted the remove-unused-import branch April 18, 2021 08:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

merge me Label to trigger pull request merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants