OSOE-110: Fix verify action that fails if the PR title has apostrophe in it

.github/actions/build-dotnet/Build-DotNetSolution.ps1

@@ -50,6 +50,12 @@ if (Test-Path src/Utilities/Lombiq.Gulp.Extensions/Lombiq.Gulp.Extensions.csproj
     Write-Output "::endgroup::"
 }
 
+# This prepares the solution with the Lombiq.Analyzers files. The output and exit code are dismissed because they will
+# be in error if there is a project without the LombiqNetAnalyzers target. Then there is nothing to do, and the target 
+# will still run on the projects that have it.
+dotnet msbuild '-target:Restore;LombiqNetAnalyzers' $Solution | Out-Null
+bash -c 'true'
+
 Write-Output "Building solution with ``dotnet build $Solution $($buildSwitches -join " ")``."
 
 $errorLines = New-Object "System.Collections.Generic.List[string]"
@@ -72,6 +78,15 @@ if ($expectedErrorCodes)
     $fail = 0
     $report = New-Object "System.Text.StringBuilder" "`n"
 
+    if ($null -eq $errorCodes -or -not $errorCodes.Count)
+    {
+        $expectedCount = $expectedErrorCodes.Count
+        $expectedCodesJoined = $expectedErrorCodes -join ', '
+
+        Write-Output "::error::Expected $expectedCount error codes ($expectedCodesJoined), but none were displayed."
+        exit 1
+    }
+
     $length = [System.Math]::Max($errorCodes.Count, $expectedErrorCodes.Count)
     foreach ($index in 0..($length - 1))
     {

.github/actions/publish-nuget/New-NugetPackages.ps1

@@ -0,0 +1,18 @@
+param([array] $Arguments)
+
+foreach ($project in (dotnet sln list | Select-Object -Skip 2 | % { Get-ChildItem $_ }))
+{
+    Push-Location $project.Directory
+
+    $nuspecFile = (Get-ChildItem *.nuspec).Name
+    if ($nuspecFile.Count -eq 1)
+    {
+        dotnet pack $project -p:NuspecFile="$nuspecFile" @Arguments
+    }
+    else
+    {
+        dotnet pack $project @Arguments
+    }
+
+    Pop-Location
+}

.github/actions/publish-nuget/action.yml

@@ -55,6 +55,11 @@ runs:
       shell: pwsh
       run: dotnet restore -p:NuGetBuild=true --verbosity ${{ inputs.verbosity }}
 
+    - name: Generate nuspec file if needed
+      if: hashFiles('ConvertTo-Nuspec.ps1')
+      shell: pwsh
+      run: ./ConvertTo-Nuspec.ps1 '${{ steps.setup.outputs.publish-version }}'
+
     - name: Build
       uses: Lombiq/GitHub-Actions/.github/actions/build-dotnet@dev
       # Notes on the configuration:
@@ -82,6 +87,11 @@ runs:
             -p:DebugSymbols=true
             -p:DebugType=portable
 
+    - name: Actions prior to dotnet pack
+      if: hashFiles('Invoke-BeforePack.ps1')
+      shell: pwsh
+      run: ./Invoke-BeforePack.ps1
+
     - name: Pack
       shell: pwsh
       # Notes on the configuration apart from what's also for dotnet build:
@@ -92,19 +102,20 @@ runs:
       # * -p:IncludeSymbols=true -p:SymbolPackageFormat=snupkg are needed to generate symbol packages:
       #   https://docs.microsoft.com/en-us/nuget/create-packages/symbol-packages-snupkg.
       run: |
-        $noWarn = @('NU5104') + @'
+        $noWarn = @('NU5104', 'NU5100') + @'
         ${{ inputs.dotnet-pack-ignore-warning }}
         '@.Split() | ? { $_ }
 
-        $arguments = @(
+        New-NugetPackages @(
             "--configuration:Release",
             "--warnaserror",
             "--no-restore",
             "--no-build",
-            "--output:artifacts",
+            "--output:" + (Join-Path $PWD artifacts),
             "--verbosity:${{ inputs.verbosity }}",
             "-p:NuGetBuild=true",
             "-p:Version=${{ steps.setup.outputs.publish-version }}",
+            "-p:NuspecProperties=version=${{ steps.setup.outputs.publish-version }}",
             "-p:GenerateDocumentationFile=True",
             "-p:NoWarn=$($noWarn -join '%3B')",
             "-p:TreatWarningsAsErrors=true",
@@ -113,7 +124,6 @@ runs:
             "-p:SymbolPackageFormat=snupkg",
             "-p:NoDefaultExcludes=true"
         )
-        dotnet pack @arguments
 
     - name: Push with dotnet
       shell: pwsh

.github/actions/verify-submodule-pull-request/action.yml

@@ -25,10 +25,16 @@ runs:
         New-Item -ItemType File -Force $Profile | Out-Null
         Get-Content -Raw ${{ github.action_path }}/functions.ps1 >> $Profile
 
+    # We are using env for the title input in these steps because it realistically could contain 
+    # quotes apostrophes or even both. This way those don't need to be escaped.
     - name: Ensure Current PR Title Includes Issue Code
       shell: pwsh
-      run: Check-Current '${{ inputs.title }}'
+      env:
+        PR_TITLE: ${{ inputs.title }}
+      run: Check-Current $Env:PR_TITLE
 
     - name: Ensure Parent Repository Contains Matching PR
       shell: pwsh
-      run: Check-Parent '${{ inputs.repository }}' '${{ inputs.title }}'
+      env:
+        PR_TITLE: ${{ inputs.title }}
+      run: Check-Parent '${{ inputs.repository }}' $Env:PR_TITLE

.github/workflows/publish-nuget.yml

@@ -56,7 +56,7 @@ jobs:
         dotnet-version: ${{ inputs.dotnet-version }}
 
     - name: Publish to NuGet
-      uses: Lombiq/GitHub-Actions/.github/actions/publish-nuget@dev
+      uses: Lombiq/GitHub-Actions/.github/actions/publish-nuget@issue/OSOE-110
       with:
         source: ${{ inputs.source }}
         verbosity: ${{ inputs.verbosity }}

.github/workflows/test-analysis-failure.yml

@@ -87,7 +87,7 @@ jobs:
       uses: Lombiq/GitHub-Actions/.github/actions/enable-corepack@dev
 
     - name: Build and Static Code Analysis
-      uses: Lombiq/GitHub-Actions/.github/actions/build-dotnet@dev
+      uses: Lombiq/GitHub-Actions/.github/actions/build-dotnet@issue/OSOE-110
       with:
         directory: ${{ inputs.build-directory }}
         verbosity: quiet

.github/workflows/verify-submodule-pull-request.yml

@@ -13,6 +13,6 @@ jobs:
     runs-on: ubuntu-latest
     steps:
     - name: Ensure Parent PR Exists
-      uses: Lombiq/GitHub-Actions/.github/actions/verify-submodule-pull-request@dev
+      uses: Lombiq/GitHub-Actions/.github/actions/verify-submodule-pull-request@issue/OSOE-110
       with:
         title: ${{ github.event.pull_request.title }}