Set extra publish properties

jeremytammik · Sep 18, 2024 · e66ddc2 · e66ddc2
1 parent 600edb4
commit e66ddc2
Show file tree

Hide file tree

Showing 3 changed files with 22 additions and 27 deletions.
diff --git a/.github/workflows/Tools.yml b/.github/workflows/Tools.yml
@@ -14,9 +14,9 @@ jobs:
           fetch-depth: 0
       - name: Run Nuke Build
         working-directory: ./tools
-        run: ./.nuke/build.cmd CreateInstaller Publish --GitHubToken ${{ secrets.GITHUB_TOKEN }}
+        run: ./.nuke/build.cmd CreateInstaller Publish
       - name: Upload artifacts
         uses: actions/upload-artifact@v4
         with:
           name: Revit-tools
-          path: tools/output
+          path: tools/output/**/*
diff --git a/tools/build/Build.Publish.cs b/tools/build/Build.Publish.cs
@@ -9,19 +9,30 @@ sealed partial class Build
         {
             DotNetPublish(settings => settings
                 .SetProject(Solution.DependenciesReport.Path)
-                .SetOutput("output")
-                .SetRuntime("win-x64")
+                .SetOutput("output/native")
+                .SetCommonPublishProperties()
                 .SetProperty("PublishAot", true)
-                .SetProperty("AssemblyName", "DependenciesReport-native")
-                .SetVerbosity(DotNetVerbosity.minimal));
-
+                .SetProperty("AssemblyName", "DependenciesReport-native"));
+
             DotNetPublish(settings => settings
                 .SetProject(Solution.DependenciesReport.Path)
-                .SetOutput("output")
-                .SetRuntime("win-x64")
+                .SetOutput("output/dotnet")
+                .SetCommonPublishProperties()
                 .SetSelfContained(false)
                 .SetPublishSingleFile(true)
-                .SetProperty("AssemblyName", "DependenciesReport-dotnet8")
-                .SetVerbosity(DotNetVerbosity.minimal));
+                .SetProperty("AssemblyName", "DependenciesReport-dotnet"));
         });
+}
+
+static class PublishExtensions
+{
+    public static DotNetPublishSettings SetCommonPublishProperties(this DotNetPublishSettings settings)
+    {
+        return settings
+            .SetRuntime("win-x64")
+            .SetConfiguration("Release")
+            .SetProperty("DebugType", "None")
+            .SetProperty("DebugSymbols", "False")
+            .SetVerbosity(DotNetVerbosity.minimal);
+    }
 }
diff --git a/tools/source/DependenciesReport/DependenciesReport.sln b/tools/source/DependenciesReport/DependenciesReport.sln