-
Notifications
You must be signed in to change notification settings - Fork 205
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
Update hlint #5123
Update hlint #5123
Conversation
d8ca9c7
to
a629019
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I’m not a fan of the rules encouraging point-free style. Can we please disable them by default.
a629019
to
08f0491
Compare
@cocreature PR adjusted as per your comments. |
08f0491
to
4618e08
Compare
changelog_begin changelog_end
4618e08
to
ef219f1
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
@@ -1,6 +1,8 @@ | |||
-- Copyright (c) 2020, Digital Asset (Switzerland) GmbH and/or its affiliates. | |||
-- All rights reserved. | |||
|
|||
-- @INFO Use catOptionals | |||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What are we testing here? And why is this test called ActionFail
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That's lost in the mists of time but if I had to guess, it's that fail
works correctly in the presence of RebindableSyntax
(ghc-proposals/ghc-proposals#168).
Update hlint. Depends on digital-asset/hlint#52