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

Document what is required to get HLint working #249

Closed
ndmitchell opened this issue Jul 28, 2020 · 5 comments
Closed

Document what is required to get HLint working #249

ndmitchell opened this issue Jul 28, 2020 · 5 comments

Comments

@ndmitchell
Copy link
Collaborator

A search of the repo suggests that HLint is supported. But on my sample project, it doesn't make any suggestions, even when I define the classic:

foo f g x = map f (map g x)

(That was the first hint HLint ever matched.) The docs don't mention HLint at all. It would be useful if the docs included a note that HLint exists, some way of testing it, and ideally some way of validating that it works.

Or maybe HLint isn't yet supported?

@fendor
Copy link
Collaborator

fendor commented Jul 28, 2020

HLint is not supported yet, see the PR #166 to add it.

@Ailrun
Copy link
Member

Ailrun commented Jul 28, 2020

#32 is the issue tracking the HLint plugin.

@lukel97
Copy link
Collaborator

lukel97 commented Jul 28, 2020

Where is hlint being advertised? We should probably take that down until it's actually merged. I can imagine it might be a leftover of code being copied over from HIE

@ndmitchell
Copy link
Collaborator Author

It wasn't being advertised - I just searched for HLint and found some configuration settings. I don't think there's any advertising removal that needs doing, but lets hope #166 is ready soon :)

@jneira
Copy link
Member

jneira commented Sep 9, 2020

@ndmitchell i think we can close this one as we have the main issue and the ongoing pr will come soon (hopefully 😄)

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

5 participants