Merge pull request #3450 from NativeScript/fatme/print-messages

Fatme · web-flow · commit 24b446717112 · 2018-03-13T14:48:08.000+02:00
Don't print info messages when environment is correctly configured.
diff --git a/lib/services/doctor-service.ts b/lib/services/doctor-service.ts
@@ -74,9 +74,13 @@ class DoctorService implements IDoctorService {
 		if (platform) {
 			infos = this.filterInfosByPlatform(infos, platform);
 		}
-		this.printInfosCore(infos);
 
 		const warnings = this.filterInfosByType(infos, constants.WARNING_TYPE_NAME);
+		if (warnings.length > 0) {
+			this.printInfosCore(infos);
+		} else {
+			infos.map(info => this.$logger.trace(info.message));
+		}
 		return warnings.length === 0;
 	}
 

Original file line number	Diff line number	Diff line change
`@@ -74,9 +74,13 @@ class DoctorService implements IDoctorService {`
`74`	`74`	`if (platform) {`
`75`	`75`	`infos = this.filterInfosByPlatform(infos, platform);`
`76`	`76`	`}`
`77`		`- this.printInfosCore(infos);`
`78`	`77`
`79`	`78`	`const warnings = this.filterInfosByType(infos, constants.WARNING_TYPE_NAME);`
	`79`	`+ if (warnings.length > 0) {`
	`80`	`+ this.printInfosCore(infos);`
	`81`	`+ } else {`
	`82`	`+ infos.map(info => this.$logger.trace(info.message));`
	`83`	`+ }`
`80`	`84`	`return warnings.length === 0;`
`81`	`85`	`}`
`82`	`86`