Skip to content

check-exercises.hs redundant import warning after GHC 7.10.x #122

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
petertseng opened this issue Apr 7, 2016 · 1 comment · Fixed by #154
Closed

check-exercises.hs redundant import warning after GHC 7.10.x #122

petertseng opened this issue Apr 7, 2016 · 1 comment · Fixed by #154

Comments

@petertseng
Copy link
Member

Travis keeps warning with:

_test/check-exercises.hs:18:1: warning: [-Wunused-imports]
    The import of ‘Control.Applicative’ is redundant
      except perhaps to import instances from ‘Control.Applicative’
    To import instances alone, use: import Control.Applicative()

Should check whether that's required. If not required, remove the line completely. If required, add the ().

@rbasso
Copy link
Contributor

rbasso commented Jun 17, 2016

This warning is cause by Prelude exporting some functions from Control.Applicative after GHC version 7.10.x.

If we remove this line,check-exercises.hs it will not compile in older versions:

    Not in scope: `<$>'
    Perhaps you meant `</>' (imported from System.FilePath)

Functor-Applicative-Monad Proposal
GHC 7.10.x Migration Guide

The last link has some solutions to avoid this warning.

@rbasso rbasso changed the title check-exercises.hs possibly has unused import check-exercises.hs show *redundant import*'s warning after GHC 7.10.x Jun 17, 2016
@rbasso rbasso changed the title check-exercises.hs show *redundant import*'s warning after GHC 7.10.x check-exercises.hs redundant import warning after GHC 7.10.x Jun 17, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants