Correct manual internal builds

- need to explicitly set `$(BuildNumber)` property in all build steps

nit: Run all test jobs in internal pull requests

Author: dougbu

.azure/pipelines/ci.yml

@@ -37,22 +37,25 @@ jobs:
     jobName: Windows_Build
     jobDisplayName: "Build and test: Windows"
     agentOs: Windows
+    buildArgs: $(BuildScriptArgs)
     codeSign: true
     beforeBuild:
     - powershell: "& ./src/Servers/IIS/tools/UpdateIISExpressCertificate.ps1"
       displayName: Setup IISExpress test certificates
-# Unix test jobs only run on public CI builds
-- ${{ if eq(variables['System.TeamProject'], 'public') }}:
+# Unix test jobs only run on public CI builds and PRs
+- ${{ if or(eq(variables['System.TeamProject'], 'public'), in(variables['Build.Reason'], 'PullRequest')) }}:
   - template: jobs/default-build.yml
     parameters:
       jobName: MacOs_Build
       jobDisplayName: "Build and test : MacOS"
       agentOs: MacOS
+      buildArgs: $(BuildScriptArgs)
   - template: jobs/default-build.yml
     parameters:
       jobName: Linux_Build
       jobDisplayName: "Build and test : Linux"
       agentOs: Linux
+      buildArgs: $(BuildScriptArgs)
 
 - ${{ if eq(variables['System.TeamProject'], 'internal') }}:
   - job: Windows_SharedFx
@@ -468,7 +471,7 @@ jobs:
 
   - job: SharedFX_Installers
     displayName: Build SharedFX Installers
-    dependsOn: 
+    dependsOn:
       - Linux_SharedFx
       - Linux_Musl_SharedFx
       - MacOs_SharedFx
@@ -554,7 +557,7 @@ jobs:
 
   - job: Publish
     displayName: Publish
-    dependsOn: 
+    dependsOn:
       - Windows_Installers
       - SharedFX_Installers
       - Package_Archive