Fix workspace diagnostics not respecting ignore globs for dependents

Fixes #43
JohnnyMorganz · Jun 20, 2022 · 992ca70 · 992ca70
1 parent 8940094
commit 992ca70
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -26,6 +26,7 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/).
 - Fixed equality comparison between enum items raising a type error
 - Fixed autocompletion of properties with spaces not correctly being converted into a `["property"]` index leading to a type error
 - Fixed function stringification when using an expression index call such as `data["property"]()`
+- Fixed workspace diagnostics not respecting ignore globs for dependent files
 
 ## [1.4.0] - 2022-06-12
 

diff --git a/src/LanguageServer.cpp b/src/LanguageServer.cpp
@@ -417,7 +417,8 @@ void LanguageServer::onDidChangeTextDocument(const lsp::DidChangeTextDocumentPar
                 if (filePath)
                 {
                     auto uri = Uri::file(*filePath);
-                    if (uri != params.textDocument.uri && !contains(diagnostics.relatedDocuments, uri.toString()))
+                    if (uri != params.textDocument.uri && !contains(diagnostics.relatedDocuments, uri.toString()) &&
+                        !workspace->isIgnoredFile(*filePath, config))
                     {
                         auto dependencyDiags = workspace->documentDiagnostics(lsp::DocumentDiagnosticParams{{uri}});
                         diagnostics.relatedDocuments.emplace(uri.toString(),