Skip to content

Commit 40a44d5

Browse files
committed
CI: start db container before build so it warms up during build
Moving docker compose up ahead of dotnet build lets the database container start up in parallel with the build, so it is more likely ready by the time EnsureDatabaseConnections and the tests run (helps the slower SqlServer container in particular). EnsureDatabaseConnections still gates the tests via its internal retry loop, so behaviour is unchanged.
1 parent d0ad047 commit 40a44d5

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

.github/workflows/dotnet.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -33,9 +33,9 @@ jobs:
3333
- uses: actions/setup-dotnet@v5
3434
with:
3535
dotnet-version: '10.0.x'
36-
- run: dotnet build Cleipnir.ResilientFunctions.sln
3736
- if: matrix.compose-service != ''
3837
run: docker compose up -d ${{ matrix.compose-service }}
38+
- run: dotnet build Cleipnir.ResilientFunctions.sln
3939
- if: matrix.compose-service != ''
4040
run: dotnet run --project ./Stores/EnsureDatabaseConnections/EnsureDatabaseConnections.csproj --no-build -- ${{ matrix.db-name }}
4141
- run: dotnet test ./${{ matrix.test-project }} --no-build --logger "console;verbosity=detailed"

0 commit comments

Comments
 (0)