Merge pull request #3761 from NativeScript/fatme/move-skipping-node_modules-msg

Do not show "Skipping node_modules folder!..." message from Sidekick

Author: Fatme

lib/common

@@ -11,7 +11,8 @@ export class LiveSyncCommandHelper implements ILiveSyncCommandHelper {
 		private $analyticsService: IAnalyticsService,
 		private $bundleValidatorHelper: IBundleValidatorHelper,
 		private $errors: IErrors,
-		private $iOSSimulatorLogProvider: Mobile.IiOSSimulatorLogProvider) {
+		private $iOSSimulatorLogProvider: Mobile.IiOSSimulatorLogProvider,
+		private $logger: ILogger) {
 		this.$analyticsService.setShouldDispose(this.$options.justlaunch || !this.$options.watch);
 	}
 
@@ -21,6 +22,10 @@ export class LiveSyncCommandHelper implements ILiveSyncCommandHelper {
 	}
 
 	public async executeCommandLiveSync(platform?: string, additionalOptions?: ILiveSyncCommandHelperAdditionalOptions) {
+		if (!this.$options.syncAllFiles) {
+			this.$logger.info("Skipping node_modules folder! Use the syncAllFiles option to sync files from this folder.");
+		}
+
 		const emulator = this.$options.emulator;
 		await this.$devicesService.initialize({
 			deviceId: this.$options.device,

lib/helpers/livesync-command-helper.ts

@@ -35,7 +35,6 @@ export class IOSLiveSyncService extends PlatformLiveSyncServiceBase implements I
 		this.$fs.copyFile(path.join(path.dirname(projectFilesPath), `${APP_FOLDER_NAME}/*`), tempApp);
 
 		if (!syncInfo.syncAllFiles) {
-			this.$logger.info("Skipping node_modules folder! Use the syncAllFiles option to sync files from this folder.");
 			this.$fs.deleteDirectory(path.join(tempApp, TNS_MODULES_FOLDER_NAME));
 		}