You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Describe the bug
When the file is set to the sourceName, coc will display the file path as the diagnostic source, which is useless.
Furthermore, when sourceNameFilter is set, this prevents the display of diagnostics because it filters the diagnostic source with the file name. A separate filename/filepath entry should be used instead for such a filter.
The source is the source of the diagnostic, not the file that is diagnosed, as such the source for example should be "shellcheck ${level}" and not the the json path to the "file" field.
But this can't be used because sourceName checks the json object's path for the provided string instead of interpreting it like message does, causing the parsing to fail.
Curiously "spellcheck" is the source, set correctly, when sourceName is not provided.
A sourceNameFilter should be renamed to fileNameFilter, and an additional element should be added to the parsers to get the file name if a filter is to be used on that.
I would then like to do something like
[...]
"file": "file",
"fileNameFilter": true,
"security": "level", // should be severity, by the way"sourceName": "shellcheck ${level}"
[...]
Screenshots
Desktop (please complete the following information):
Describe the bug
When the file is set to the sourceName, coc will display the file path as the diagnostic source, which is useless.
Furthermore, when sourceNameFilter is set, this prevents the display of diagnostics because it filters the diagnostic source with the file name. A separate filename/filepath entry should be used instead for such a filter.
The source is the source of the diagnostic, not the file that is diagnosed, as such the source for example should be "shellcheck ${level}" and not the the json path to the "file" field.
But this can't be used because sourceName checks the json object's path for the provided string instead of interpreting it like message does, causing the parsing to fail.
Curiously "spellcheck" is the source, set correctly, when sourceName is not provided.
To Reproduce
Steps to reproduce the behavior
Expected behavior
A sourceNameFilter should be renamed to fileNameFilter, and an additional element should be added to the parsers to get the file name if a filter is to be used on that.
I would then like to do something like
Screenshots
Desktop (please complete the following information):
Log:
diagnostics-languageserver.log
The text was updated successfully, but these errors were encountered: