Skip version /p: args on PR builds when GitVersion step is skipped

The Compute version properties step is gated by 'github.event_name != pull_request', so on PR runs steps.version.outputs.* are empty. Previously the dotnet build step still passed /p:Version=, /p:FileVersion=, /p:InformationalVersion= with empty values. Wrap each /p:* arg in a guard that omits the property when the corresponding step output is empty.

Author: mkrueger

.github/workflows/validate-and-package.yml

@@ -142,9 +142,9 @@ jobs:
           dotnet build CosmosDBShell.sln
           --configuration $env:BUILD_CONFIGURATION
           --no-restore
-          /p:Version=${{ steps.version.outputs.assembly_version }}
-          /p:FileVersion=${{ steps.version.outputs.file_version }}
-          /p:InformationalVersion=${{ steps.version.outputs.informational_version }}
+          $(if ('${{ steps.version.outputs.assembly_version }}') { "/p:Version=${{ steps.version.outputs.assembly_version }}" })
+          $(if ('${{ steps.version.outputs.file_version }}') { "/p:FileVersion=${{ steps.version.outputs.file_version }}" })
+          $(if ('${{ steps.version.outputs.informational_version }}') { "/p:InformationalVersion=${{ steps.version.outputs.informational_version }}" })
         shell: pwsh
 
       - name: Test solution