Skip to content

Commit

Permalink
test: fix 'npm test' on Apple silicon Macs (#3939)
Browse files Browse the repository at this point in the history
Specifying the mssql image platform is necessary on *arm* Macs
because the mssql Docker image only provides binaries for the
amd64 architecture. My understanding is that Rosetta emulation
on recent macOS supports running the amd binary just fine.
  • Loading branch information
trentm authored Mar 25, 2024
1 parent 42eef29 commit b84e303
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 0 deletions.
1 change: 1 addition & 0 deletions .ci/docker/docker-compose.yml
Original file line number Diff line number Diff line change
Expand Up @@ -32,6 +32,7 @@ services:

mssql:
image: mcr.microsoft.com/mssql/server
platform: linux/amd64
environment:
- ACCEPT_EULA=Y
- SA_PASSWORD=Very(!)Secure
Expand Down
1 change: 1 addition & 0 deletions test/docker-compose.yml
Original file line number Diff line number Diff line change
Expand Up @@ -32,6 +32,7 @@ services:

mssql:
image: mcr.microsoft.com/mssql/server
platform: linux/amd64
environment:
- ACCEPT_EULA=Y
- SA_PASSWORD=Very(!)Secure
Expand Down

0 comments on commit b84e303

Please sign in to comment.