#9 Remove the cli argument of external configuration file when it's not necessary

Author: xabikos

src/Webpack/ArgumentsHelper.cs

@@ -13,8 +13,11 @@ internal class ArgumentsHelper {
 		/// <summary>
 		/// Creates and returns the appropriate arguments list for the webpack based on the provided options
 		/// </summary>
-		public static string GetWebpackArguments(string rootPath, WebpackOptions options) {
-			var result = new StringBuilder(DefaultDevFile);
+		public static string GetWebpackArguments(string rootPath, WebpackOptions options, bool includeDefaultConfigFile) {
+			var result = new StringBuilder();
+			if(includeDefaultConfigFile) {
+				result.Append(DefaultDevFile);
+			}
 			if (options.HandleStyles && options.StylesTypes.Any()) {
 				if (options.StylesTypes.Contains(StylesType.Css)) {
 					result.Append(CssFiles);

src/Webpack/Webpack.cs

@@ -30,11 +30,11 @@ public WebPackMiddlewareOptions Execute(WebpackOptions options) {
 			EnsuereNodeModluesInstalled(options.EnableHotLoading, logger);
 			logger.LogInformation($"All node modules are properly installed");
 
-			CreateWebpackConfigurationFile(options);
+			var includeDefaultConfigFile = CreateWebpackConfigurationFile(options);
 
 			logger.LogInformation($"{toolToExecute} Execution started");
 			try {
-				var arguments = ArgumentsHelper.GetWebpackArguments(_webRootPath, options);
+				var arguments = ArgumentsHelper.GetWebpackArguments(_webRootPath, options, includeDefaultConfigFile);
 				logger.LogInformation($"{toolToExecute} is called with these arguments: {arguments}");
 				Process process = new Process();
 				process.StartInfo = new ProcessStartInfo() {
@@ -109,10 +109,7 @@ private static string GetNodeExecutable(string module) {
 			return executable;
 		}
 
-		private static void CreateWebpackConfigurationFile(WebpackOptions options) {
-			if (!Directory.Exists("webpack")) {
-				Directory.CreateDirectory("webpack");
-			}
+		private static bool CreateWebpackConfigurationFile(WebpackOptions options) {
 			var presets = new List<string>() {
 				"es2015"
 			};
@@ -146,6 +143,9 @@ private static void CreateWebpackConfigurationFile(WebpackOptions options) {
 			};
 			// Create the external configuration file only if we need to use babel-loader
 			if (loaders.Count > 0) {
+				if (!Directory.Exists("webpack")) {
+					Directory.CreateDirectory("webpack");
+				}
 				var jsonResult = JsonConvert.SerializeObject(exports,
 					new JsonSerializerSettings {
 						Formatting = Formatting.Indented,
@@ -157,7 +157,9 @@ private static void CreateWebpackConfigurationFile(WebpackOptions options) {
 						streamWriter.WriteLine(fileContent);
 					}
 				}
+				return true;
 			}
+			return false;
 		}
 
 	}