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

Hlint #2

Merged
merged 2 commits into from Dec 19, 2014

Conversation

Projects
None yet
2 participants
@fffej
Contributor

fffej commented Dec 19, 2014

Hi, saw your post on HN. I love the Norvig spell checker; great to see small lumps of code that give a whole lot of power.

As a fellow newbie Haskeller, Have you seen hlint (http://community.haskell.org/~ndm/hlint/)?

It's a linter for Haskell and gives lots of suggestions. This pull request applies some of those (mostly removing some redundant () and $). There's also the handy function comparing from Data.Ord which simplifies the lambda in the sortBy function.

@MarcoSero

This comment has been minimized.

Show comment
Hide comment
@MarcoSero

MarcoSero Dec 19, 2014

Owner

This is great!
Thanks for the feedback and for the time you spent putting this together.

Owner

MarcoSero commented Dec 19, 2014

This is great!
Thanks for the feedback and for the time you spent putting this together.

MarcoSero added a commit that referenced this pull request Dec 19, 2014

Merge pull request #2 from fffej/hlint
Using Hlint to remove redundant parenthesis

@MarcoSero MarcoSero merged commit 3eacf71 into MarcoSero:master Dec 19, 2014

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