Migrate change-type-signature-plugin to use structured diagnostics#4632
Migrate change-type-signature-plugin to use structured diagnostics#4632mergify[bot] merged 4 commits intohaskell:masterfrom
Conversation
5556c3e to
be06dc8
Compare
fendor
left a comment
There was a problem hiding this comment.
LGTM, thank you for taking the time and modernising the whole plugin!
Small comments, otherwise good to go, imo
| matchingDiagnostic :: ErrInfo -> Maybe DeclName | ||
| matchingDiagnostic ErrInfo{errInfoContext} = |
There was a problem hiding this comment.
We still need that? Should we open a GHC issue for adding the DeclName context to the error message?
There was a problem hiding this comment.
We still need that?
Sadly, I think we do
Should we open a GHC issue for adding the
DeclNamecontext to the error message?
That would be awesome
There was a problem hiding this comment.
@fendor do you want me to attempt to open a GHC issue for this?
|
Stack CI test failure is genuine, otherwise, let's merge! |
92bbd8e to
e25716f
Compare
|
This pull request has been removed from the queue for the following reason: The merge conditions cannot be satisfied due to failing checks:You may have to fix your CI before adding the pull request to the queue again. |
|
@Mergifyio requeue |
❌ Command disallowed due to command restrictions in the Mergify configuration.Details
|
This addresses using structured diagnostics from GHC (rather than regular expressions) for
hls-change-type-signature-plugin.A couple of notes:
Development.IDE.GHC.Compat.Error?