Fix comments

microsoft · Aug 27, 2024 · e6c3331 · e6c3331
1 parent d7bc960
commit e6c3331
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 2 deletions.
diff --git a/azure-pipelines/1esmain.yml b/azure-pipelines/1esmain.yml
@@ -35,5 +35,4 @@ extends:
       - template: ./1esstages.yml
         parameters:
           useAzureFederatedCredentials: ${{ parameters.useAzureFederatedCredentials }}
-          additionalSetupSteps: ${{ parameters.additionalSetupSteps }}
-          # Enable the signing toolset when the build is not a PR and the branch is main or starts with rel/
+          additionalSetupSteps: ${{ parameters.additionalSetupSteps }}
diff --git a/azure-pipelines/1esstages.yml b/azure-pipelines/1esstages.yml
@@ -19,6 +19,7 @@ stages:
             templateContext:
               mb: # Enable the MicroBuild Signing toolset
                 signing:
+                  # Enable the signing toolset when the build is not a PR and the branch is main or starts with rel/
                   enabled: $[and(ne(variables['Build.Reason'], 'PullRequest'), or(eq(variables['Build.SourceBranchName'], 'main'), startsWith(variables['Build.SourceBranchName'], 'rel/')))]
                   signType: real # options are 'real' & 'test'
                   zipSources: false

diff --git a/azure-pipelines/templates/sign.yml b/azure-pipelines/templates/sign.yml
@@ -11,6 +11,7 @@ steps:
           Write-Output "SignExtension.signproj file not found. Skipping signing."
       }
     displayName: "\U0001F449 Check for SignExtension.signproj File"
+    # Run this step when the build is not a PR and the branch is main or starts with rel/
     condition: and(ne(variables['Build.Reason'], 'PullRequest'), or(eq(variables['Build.SourceBranchName'], 'main'), startsWith(variables['Build.SourceBranchName'], 'rel/')))
 
   # put the extension name and version from the package.json into variables to use later. Variables can be used in later steps as $(package.name) and $(package.version)