Skip to content
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

Haskell hlint : HLint.hs not taking into account #1853

Closed
PierreR opened this issue Jun 4, 2015 · 6 comments
Closed

Haskell hlint : HLint.hs not taking into account #1853

PierreR opened this issue Jun 4, 2015 · 6 comments

Comments

@PierreR
Copy link
Contributor

PierreR commented Jun 4, 2015

I am trying to disable some hlint errors by using a HLint.hs file at the root of the project:

import "hint" HLint.HLint

ignore "Redundant lambda"

Unfortunately it does not work. I stil get a red point on:

let queryfunc = \node -> ....

On the command line if I use hlint it does work (it is telling me a suggestion has been ignored).

Is there a way to disable these hlint errors ?

At least it would be handy to use another color than red to show the hlint "errors" (the code compiles just fine after all).

@syl20bnr
Copy link
Owner

syl20bnr commented Jun 5, 2015

Related to #753

@d12frosted
Copy link
Contributor

@PierreR what does running hlint path/to/file returns? Does it say anything about lambda?

By the way, for me to disable hlints about lambda I need to put following into HLint.hs:

import "hint" HLint.HLint

ignore "Avoid lambda"
ignore "Redundant lambda"

@PierreR
Copy link
Contributor Author

PierreR commented Oct 26, 2015

@d12frosted As already described, hlint path/to/file is working fine (it is telling me a suggestion has been ignored).

Do you want me to test it again ? I haven't done so for a while so it might be fixed now.

@d12frosted
Copy link
Contributor

@PierreR oh, right you already said that it's working when invoking from terminal. But yeah, could you please test again? Because I remember previously it didn't work for me in Emacs, but it's working.

@PierreR
Copy link
Contributor Author

PierreR commented Oct 27, 2015

Yep this is working now. Thanks.

@PierreR PierreR closed this as completed Oct 27, 2015
@d12frosted
Copy link
Contributor

Great! Good to hear 😄

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

No branches or pull requests

3 participants