[deploy] disable zipping of assets by default. Addresses #93.

Author: stefanloerwald

README.md

@@ -23,6 +23,11 @@ As a benefit, this implementation is cross-platform (x64 linux/win are tested, p
 
 ### Changelog
 
+#### Changes in version 4.X.Y
+
+The default value for zipping minified assets has changed from `true` to `false`. This has no effect to pre-existing configuration files.
+The change was made in anticipation of a new feature in the net9.0 SDK.
+
 #### Changes in version 3.8.14
 
 Support for net7.0 was dropped. The nupkg file was getting outrageously large due to dependencies and duplication across the target frameworks.

Tests_WebCompiler/AutoPipelineRealJustZipTests.cs

@@ -15,6 +15,7 @@ public void CreatePipeline()
             {
                 Minifiers = new MinificationSettings
                 {
+                    GZip = true,
                     Css = new CssMinifySettings
                     {
                         TermSemicolons = false

Tests_WebCompiler/AutoPipelineRealTests.cs

@@ -15,6 +15,7 @@ public void CreatePipeline()
             {
                 Minifiers = new MinificationSettings
                 {
+                    GZip = true,
                     Css = new CssMinifySettings
                     {
                         TermSemicolons = true

Tests_WebCompiler/AutoPipelineScssTests.cs

@@ -15,6 +15,7 @@ public void CreatePipeline()
             {
                 Minifiers = new MinificationSettings
                 {
+                    GZip = true,
                     Css = new CssMinifySettings
                     {
                         TermSemicolons = false

Tests_WebCompiler/Tests_WebCompiler.csproj

@@ -6,7 +6,7 @@
     <RootNamespace>Tests_WebCompiler</RootNamespace>
   </PropertyGroup>
   <ItemGroup>
-    <PackageReference Include="NUnit" Version="4.2.1" />
+    <PackageReference Include="NUnit" Version="4.2.2" />
     <PackageReference Include="NUnit3TestAdapter" Version="4.6.0">
       <PrivateAssets>all</PrivateAssets>
       <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>

Tests_WebCompiler/WholeProgramTests.cs

@@ -353,7 +353,7 @@ public void OutputAtSamePlace()
                 }
             }
             DeleteTemporaryFiles();
-            Assert.That(Program.Main("../../../TestCases/Js/test.js", "../../../TestCases/MinCss/site.min.css", "-r", "../../../TestCases/Css"), Is.EqualTo(0));
+            Assert.That(Program.Main("../../../TestCases/Js/test.js", "../../../TestCases/MinCss/site.min.css", "-r", "../../../TestCases/Css", "-z", "enable"), Is.EqualTo(0));
             foreach (var output_file in output_files)
             {
                 Assert.That(File.Exists(output_file), $"Output {output_file} should exist");
@@ -397,7 +397,7 @@ public void OutputAtSamePlaceUnnecessarilySpecified()
                 }
             }
             DeleteTemporaryFiles();
-            Assert.That(Program.Main("../../../TestCases/Js/test.js", "../../../TestCases/MinCss/site.min.css", "-r", "../../../TestCases/Css", "-o", "../../../TestCases"), Is.EqualTo(0));
+            Assert.That(Program.Main("../../../TestCases/Js/test.js", "../../../TestCases/MinCss/site.min.css", "-r", "../../../TestCases/Css", "-o", "../../../TestCases", "-z", "enable"), Is.EqualTo(0));
             foreach (var output_file in output_files)
             {
                 Assert.That(File.Exists(output_file), $"Output {output_file} should exist");
@@ -446,7 +446,7 @@ public void OutputWithPreservation()
                 }
             }
             DeleteTemporaryFiles();
-            Assert.That(Program.Main("../../../TestCases/Js/test.js", "../../../TestCases/MinCss/site.min.css", "-r", "../../../TestCases/Css", "-o", "../../../output/path"), Is.EqualTo(0));
+            Assert.That(Program.Main("../../../TestCases/Js/test.js", "../../../TestCases/MinCss/site.min.css", "-r", "../../../TestCases/Css", "-o", "../../../output/path", "-z", "enable"), Is.EqualTo(0));
             foreach (var output_file in output_files)
             {
                 Assert.That(File.Exists(output_file), $"Output {output_file} should exist");

WebCompiler/Configuration/MinificationSettings.cs

@@ -4,7 +4,7 @@ namespace WebCompiler.Configuration
 {
     public class MinificationSettings
     {
-        public bool GZip { get; set; } = true;
+        public bool GZip { get; set; } = false;
         public bool Enabled { get; set; } = true;
         public CssMinifySettings Css { get; set; } = new CssMinifySettings();
         public JavaScriptMinifySettings Javascript { get; set; } = new JavaScriptMinifySettings();

WebCompiler/WebCompiler.csproj

@@ -10,7 +10,7 @@
   </PropertyGroup>
   <ItemGroup>
     <PackageReference Include="AutoprefixerHost" Version="3.0.36" />
-    <PackageReference Include="DartSassHost" Version="1.0.13" />
+    <PackageReference Include="DartSassHost" Version="1.0.14" />
     <PackageReference Include="JavaScriptEngineSwitcher.V8" Version="3.24.2" />
     <PackageReference Include="Microsoft.ClearScript.V8.Native.win-x86" Version="7.4.5" />
     <PackageReference Include="Microsoft.ClearScript.V8.Native.win-x64" Version="7.4.5" />

WebCompiler/package.csproj

@@ -20,7 +20,7 @@
   <!--PackageId, Version-->
   <PropertyGroup>
     <PackageId>Excubo.WebCompiler</PackageId>
-    <VersionPrefix>3.9.1</VersionPrefix>
+    <VersionPrefix>4.0.0</VersionPrefix>
     <!--<VersionSuffix>preview.1</VersionSuffix>-->
   </PropertyGroup>
   <!--Self Include-->