Skip to content

Catch GHC errors in listing module names#1367

Merged
mergify[bot] merged 5 commits into
haskell:masterfrom
berberman:module-name-completion
Feb 14, 2021
Merged

Catch GHC errors in listing module names#1367
mergify[bot] merged 5 commits into
haskell:masterfrom
berberman:module-name-completion

Conversation

@berberman

Copy link
Copy Markdown
Collaborator

See #1365

Let's catch this panic, so that other functionalities of HLS can continue to work.

Comment thread ghcide/src/Development/IDE/Plugin/Completions/Logic.hs
@berberman
berberman requested a review from pepeiborra February 14, 2021 10:49
Comment thread ghcide/src/Development/IDE/Types/HscEnvEq.hs Outdated
Comment thread ghcide/src/Development/IDE/Types/HscEnvEq.hs Outdated
Comment thread ghcide/src/Development/IDE/Types/HscEnvEq.hs Outdated
@pepeiborra

Copy link
Copy Markdown
Collaborator

GHC bug handled and performance improved, great job!

@pepeiborra pepeiborra added the merge me Label to trigger pull request merge label Feb 14, 2021
@mergify
mergify Bot merged commit 64e9525 into haskell:master Feb 14, 2021
@berberman
berberman deleted the module-name-completion branch February 15, 2021 03:20
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