ci(coverage): make ReportGenerator install idempotent + pinned (review follow-up)

Cold-review should-fix on PR #370: `dotnet tool install` errors under set -e
if the tool is already present (retried step / cached runner image), which
would silently degrade the continue-on-error summary step to a no-op. Use
`dotnet tool update` (install-or-update) pinned to 5.5.10 for reproducible
output. Also order coverlet.collector after CliWrap to keep the ItemGroup
alphabetical (nit).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

.gitea/workflows/docker-build.yml

@@ -121,7 +121,9 @@ jobs:
             exit 0
           fi
           echo "Found ${#reports[@]} coverage report(s)."
-          dotnet tool install --global dotnet-reportgenerator-globaltool >/dev/null
+          # `update` is install-or-update (idempotent, unlike `install` which errors if the tool
+          # is already present under set -e); pinned for reproducible summary output.
+          dotnet tool update --global dotnet-reportgenerator-globaltool --version 5.5.10 >/dev/null
           export PATH="$PATH:$HOME/.dotnet/tools"
           reportgenerator \
             "-reports:coverage/**/coverage.cobertura.xml" \

ErsatzTV.Core.Tests/ErsatzTV.Core.Tests.csproj

@@ -7,11 +7,11 @@
     </PropertyGroup>
 
     <ItemGroup>
+        <PackageReference Include="CliWrap" />
         <PackageReference Include="coverlet.collector">
             <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
             <PrivateAssets>all</PrivateAssets>
         </PackageReference>
-        <PackageReference Include="CliWrap" />
         <PackageReference Include="LanguageExt.Core" />
         <PackageReference Include="Microsoft.Extensions.Caching.Memory" />
         <PackageReference Include="Microsoft.Extensions.DependencyInjection" />