Fixes microsoft#107: "No ESLint configuration found." message keeps r…

…eappearing
adenflorian · Aug 30, 2016 · 59c03c9 · 59c03c9
1 parent e61c46a
commit 59c03c9
Showing 1 changed file with 27 additions and 2 deletions.
diff --git a/eslint-server/src/server.ts b/eslint-server/src/server.ts
@@ -236,11 +236,29 @@ function validate(document: TextDocument): void {
 	return connection.sendDiagnostics({ uri, diagnostics });
 }
 
+let ignoreNoConfigFound: boolean = false;
+
+interface ESLintError extends Error {
+	messageTemplate?: string;
+}
+
+function isNoConfigFoundError(err: any): boolean {
+	let candidate = err as ESLintError;
+	return candidate.messageTemplate === 'no-config-found' || candidate.message === 'No ESLint configuration found.';
+}
+
 function validateSingle(document: TextDocument): void {
 	try {
 		validate(document);
 	} catch (err) {
-		connection.window.showErrorMessage(getMessage(err, document));
+		if (isNoConfigFoundError(err)) {
+			if (!ignoreNoConfigFound) {
+				ignoreNoConfigFound = true;
+				connection.window.showErrorMessage(getMessage(err, document));
+			}
+		} else {
+			connection.window.showErrorMessage(getMessage(err, document));
+		}
 	}
 }
 
@@ -250,7 +268,14 @@ function validateMany(documents: TextDocument[]): void {
 		try {
 			validate(document);
 		} catch (err) {
-			tracker.add(getMessage(err, document));
+			if (isNoConfigFoundError(err)) {
+				if (!ignoreNoConfigFound) {
+					ignoreNoConfigFound = true;
+					tracker.add(getMessage(err, document));
+				}
+			} else {
+				tracker.add(getMessage(err, document));
+			}
 		}
 	});
 	tracker.sendErrors(connection);