Skip to content

Follow hlint suggestion: use lambda-case#11807

Open
philderbeast wants to merge 1 commit into
haskell:masterfrom
cabalism:hlint/use-lambda-case
Open

Follow hlint suggestion: use lambda-case#11807
philderbeast wants to merge 1 commit into
haskell:masterfrom
cabalism:hlint/use-lambda-case

Conversation

@philderbeast
Copy link
Copy Markdown
Collaborator

See #9110. Discharges and no longer ignores HLint's "use lambda-case" suggestion so that these will be suggested by our CI linting.

I'll squash commits before applying the merge label if this pull request is approved. I've left the styling change to module CmdHaddockProject separate for an easier review.


  • Patches conform to the coding conventions.
  • Is this a PR that fixes CI? If so, it will need to be backported to older cabal release branches (ask maintainers for directions).

Copy link
Copy Markdown
Collaborator

@jappeace jappeace left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice!

Comment thread cabal-install/src/Distribution/Client/CmdHaddockProject.hs
Comment thread cabal-install/src/Distribution/Client/CmdHaddockProject.hs
@ulysses4ever
Copy link
Copy Markdown
Collaborator

I'm borderline on lambda case: I like to use it myself a lot but I'd rather we don't force it on contributors: arguably, this raises the bar a bit, and I've seen Haskell newcomers who were deeply confused by this syntax.

But since the necessary two approvals are in and my doubts aren't backed by any real data, I won't block it.

@philderbeast philderbeast force-pushed the hlint/use-lambda-case branch from c102aa1 to d5335c2 Compare May 14, 2026 15:32
@philderbeast philderbeast added merge me Tell Mergify Bot to merge and removed attention: needs-review labels May 14, 2026
@mergify mergify Bot added the ready and waiting Mergify is waiting out the cooldown period label May 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

merge me Tell Mergify Bot to merge ready and waiting Mergify is waiting out the cooldown period

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants