Adjust sign build step

Author: damianh

.github/workflows/access-token-management-ci.yml

@@ -44,25 +44,32 @@ jobs:
     - name: Build
       env:
         SignClientSecret: ${{ secrets.SignClientSecret }}
-      run: ./build.ps1 sign
-
-    - name: Push packages to MyGet
-      run: dotnet nuget push artifacts\*.nupkg -s https://www.myget.org/F/duende_identityserver/api/v2/package -k ${{ secrets.MYGET }}
+      run: ./build.ps1
 
     - name: Test report
       id: test-report
       uses: dorny/test-reporter@v1
       if: success() || failure()    # run this step even if previous step failed
       with:
         name: test results
-        path: access-token-management/test/Tests/TestResults/Test.trx
+        path: access-token-management/test/AccessTokenManagement.Tests/TestResults/Test.trx
         reporter: dotnet-trx
         fail-on-error: true
         fail-on-empty: true
 
+    - name: Sign
+      if: (github.ref == 'refs/heads/main' || contains(github.ref, 'refs/tags/'))
+      env:
+        SignClientSecret: ${{ secrets.SignClientSecret }}
+      run: ./build.ps1 sign
+
+    - name: Push packages to MyGet
+      if: (github.ref == 'refs/heads/main' || contains(github.ref, 'refs/tags/'))
+      run: dotnet nuget push artifacts\*.nupkg -s https://www.myget.org/F/duende_identityserver/api/v2/package -k ${{ secrets.MYGET }}
+
     - name: Upload artifacts
       uses: actions/upload-artifact@v4
-      if: success() || failure()    # run this step even if previous step failed
+      if: (github.ref == 'refs/heads/main' || contains(github.ref, 'refs/tags/'))
       with:
         path: access-token-management/artifacts/*.nupkg
         compression-level: 0

access-token-management/build/Program.cs

@@ -20,8 +20,7 @@ private static class Targets
             public const string Build = "build";
             public const string Test = "test";
             public const string Pack = "pack";
-            public const string SignBinary = "sign-binary";
-            public const string SignPackage = "sign-package";
+            public const string Sign = "sign";
         }
 
         internal static async Task Main(string[] args)
@@ -64,15 +63,10 @@ internal static async Task Main(string[] args)
                 Run("dotnet", $"pack src/AccessTokenManagement.OpenIdConnect/AccessTokenManagement.OpenIdConnect.csproj -c Release -o {Directory.CreateDirectory(packOutput).FullName} --no-build --nologo");
             });
 
-            Target(Targets.SignPackage, DependsOn(Targets.Pack, Targets.RestoreTools), () =>
-            {
-                SignNuGet();
-            });
+            Target(Targets.Sign, DependsOn(Targets.RestoreTools), SignNuGet);
 
             Target("default", DependsOn(Targets.Test, Targets.Pack));
 
-            Target("sign", DependsOn(Targets.Test, Targets.SignPackage));
-
             await RunTargetsAndExitAsync(args, ex => ex is SimpleExec.ExitCodeException || ex.Message.EndsWith(envVarMissing));
         }