Skip to content

Conversation

@michaelpj
Copy link
Collaborator

@michaelpj michaelpj commented Mar 1, 2022

Witnesses #2716.

@michaelpj michaelpj requested a review from pepeiborra as a code owner March 1, 2022 17:34
, test no yes holeL60 hleInfo "hole without internal name #831"
, if ghcVersion >= GHC92 then
-- Broken on GHC 9.2 and above due to printing of uniques
test no yes holeL65 [] "hole with variable"
Copy link
Collaborator

Choose a reason for hiding this comment

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

I cannot remember what these parameters mean, but at least the 2nd or 3rd should be broken right?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

broken doesn't work there. I don't really understand how these work, so I just put an empty set of expectations...

@michaelpj michaelpj force-pushed the mpj/hover-variables branch from 840b9d6 to a8e238d Compare March 7, 2022 13:56
@michaelpj michaelpj added the merge me Label to trigger pull request merge label Mar 7, 2022
@mergify mergify bot merged commit 9f62337 into haskell:master Mar 8, 2022
July541 pushed a commit to July541/haskell-language-server that referenced this pull request Mar 30, 2022
Witnesses haskell#2716.

Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
Co-authored-by: Pepe Iborra <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

merge me Label to trigger pull request merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants