Skip to content

Commit

Permalink
Merge pull request #209: fix CI
Browse files Browse the repository at this point in the history
  • Loading branch information
roxblnfk authored Oct 22, 2024
2 parents 40f4115 + ec26778 commit 94da60a
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 8 deletions.
11 changes: 7 additions & 4 deletions .github/workflows/ci-mssql.yml
Original file line number Diff line number Diff line change
Expand Up @@ -26,18 +26,21 @@ jobs:
- php: '8.0'
extensions: pdo, pdo_sqlsrv
mssql: 'server:2017-latest'
- php: '8.0'
extensions: pdo, pdo_sqlsrv
mssql: 'server:2019-latest'
- php: '8.1'
extensions: pdo, pdo_sqlsrv
mssql: 'server:2019-latest'
odbc-version: 18
flag: "-C"
- php: '8.2'
extensions: pdo, pdo_sqlsrv
mssql: 'server:2019-latest'
odbc-version: 18
flag: "-C"
- php: '8.3'
extensions: pdo, pdo_sqlsrv
mssql: 'server:2019-latest'
odbc-version: 18
flag: "-C"

services:
mssql:
Expand All @@ -48,7 +51,7 @@ jobs:
MSSQL_PID: Developer
ports:
- 11433:1433
options: --name=mssql --health-cmd="/opt/mssql-tools/bin/sqlcmd -S localhost -U SA -P 'SSpaSS__1' -Q 'SELECT 1'" --health-interval=10s --health-timeout=5s --health-retries=3
options: --name=mssql --health-cmd="/opt/mssql-tools${{ matrix.odbc-version }}/bin/sqlcmd ${{ matrix.flag }} -S localhost -U SA -P 'SSpaSS__1' -Q 'SELECT 1'" --health-interval=10s --health-timeout=5s --health-retries=5

steps:
- name: Checkout
Expand Down
4 changes: 0 additions & 4 deletions tests/docker-compose.yml
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,6 @@ version: "3"
services:
sqlserver:
image: mcr.microsoft.com/mssql/server:2019-latest
restart: always
ports:
- "11433:1433"
environment:
Expand All @@ -14,8 +13,6 @@ services:

mysql_latest:
image: mysql:latest
restart: always
command: --mysql-native-password=ON
ports:
- "13306:3306"
environment:
Expand All @@ -25,7 +22,6 @@ services:

postgres:
image: postgres:12
restart: always
ports:
- "15432:5432"
environment:
Expand Down

0 comments on commit 94da60a

Please sign in to comment.