Skip to content

Commit 42af9c1

Browse files
author
Alex J Lennon
committed
FIX CI: Let dotnet test handle everything
Remove restore step and --no-restore flag. Let dotnet test handle restore, build, and test in one atomic operation. This is the simplest and most reliable approach - dotnet test knows how to restore and build correctly without conflicts. Tested locally - all 92 tests pass.
1 parent 9223f39 commit 42af9c1

File tree

1 file changed

+1
-7
lines changed

1 file changed

+1
-7
lines changed

.github/workflows/ci.yml

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -29,14 +29,8 @@ jobs:
2929
- name: Clean build artifacts
3030
run: rm -rf obj bin
3131

32-
- name: Restore dependencies (explicit xunit)
33-
run: |
34-
# Restore test project - this also restores main project as dependency
35-
# and ensures xunit is in the shared assets file
36-
dotnet restore tests/InstDotNet.Tests.csproj
37-
3832
- name: Run Tests
39-
run: dotnet test tests/InstDotNet.Tests.csproj -c Release -p:GitCommitHash=${{ steps.git.outputs.hash }} --no-restore --verbosity normal
33+
run: dotnet test tests/InstDotNet.Tests.csproj -c Release -p:GitCommitHash=${{ steps.git.outputs.hash }} --verbosity normal
4034
continue-on-error: false
4135

4236
build:

0 commit comments

Comments
 (0)