Skip to content
This repository has been archived by the owner on Nov 18, 2022. It is now read-only.

Remove extension-side problem matchers to remove duplicate error/warnings #178

Closed
wants to merge 1 commit into from

Conversation

jTitor
Copy link
Contributor

@jTitor jTitor commented Oct 26, 2017

(fixes #133)

The language server now directly posts diagnostics via LSP, so the extension doesn't have to do parsing anymore; this was causing #133.

…ings.

The language server now directly posts diagnostics via LSP, so parsing the server output creates duplicate entries.
@nrc
Copy link
Member

nrc commented Oct 27, 2017

I don't think this is the correct fix for #133 - Although the RLS does supply some errors of its own, these problem matchers are for when the user runs the cargo build (or similiar) tasks. If the user does that, then it is nice to have these matchers.

@Xanewok
Copy link
Member

Xanewok commented Mar 16, 2019

Agreed, it'd be good to keep them to be honest. It might be worth following vscode-go and deduplicating the diagnostics on the extension side. I'll close the PR but thank you regardless!

@Xanewok Xanewok closed this Mar 16, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Duplicate output in the "Problems" panel
4 participants