Skip to content

Commit bc5872a

Browse files
franklinicclaude
andcommitted
fix(ci): clean up sonarcloud workflow
- Remove redundant ternary in NuGet cache path (both branches returned same value) - Remove debug "List coverage reports" step 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
1 parent a99e5ea commit bc5872a

File tree

1 file changed

+1
-9
lines changed

1 file changed

+1
-9
lines changed

.github/workflows/sonarcloud.yml

Lines changed: 1 addition & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -97,11 +97,10 @@ jobs:
9797
with:
9898
dotnet-version: '8.0.x'
9999

100-
# Windows-specific NuGet cache path
101100
- name: Cache NuGet packages
102101
uses: actions/cache@v4
103102
with:
104-
path: ${{ runner.os == 'Windows' && '~/.nuget/packages' || '~/.nuget/packages' }}
103+
path: ~/.nuget/packages
105104
key: nuget-${{ runner.os }}-${{ hashFiles('**/*.csproj', '**/Directory.Build.props') }}
106105
restore-keys: |
107106
nuget-${{ runner.os }}-
@@ -175,13 +174,6 @@ jobs:
175174
run: |
176175
dotnet test -c Release --framework net8.0 --no-build --filter "Category!=GPU&Category!=Integration" --collect:"XPlat Code Coverage" --settings coverlet.runsettings --logger "trx;LogFileName=test-results-net8.trx" --results-directory ./TestResults
177176
178-
# Debug: List coverage files found
179-
- name: List coverage reports
180-
shell: powershell
181-
run: |
182-
Write-Host "Looking for coverage files..."
183-
Get-ChildItem -Path ./TestResults -Recurse | ForEach-Object { Write-Host $_.FullName }
184-
185177
- name: End SonarCloud analysis
186178
env:
187179
SONAR_TOKEN: ${{ secrets.SONAR_TOKEN }}

0 commit comments

Comments
 (0)