Skip to content

ci: also test in windows-latest macos-latest and macos-13-intel#7

Open
juarezr wants to merge 10 commits into
masterfrom
chore/test-arch
Open

ci: also test in windows-latest macos-latest and macos-13-intel#7
juarezr wants to merge 10 commits into
masterfrom
chore/test-arch

Conversation

@juarezr

@juarezr juarezr commented Apr 6, 2025

Copy link
Copy Markdown
Owner

Changes

  • Added jobs for testing this github action in windows-latest macos-latest and macos-13-intel
  • Added python script to tests connection to firebird in CI

@juarezr juarezr force-pushed the chore/test-arch branch 3 times, most recently from 7b57566 to fb2a2d0 Compare April 7, 2025 02:07
@juarezr juarezr closed this Apr 7, 2025
@juarezr juarezr reopened this Apr 8, 2025
@juarezr juarezr force-pushed the chore/test-arch branch 2 times, most recently from 0997420 to 2b6f540 Compare April 8, 2025 16:02
@mariuz

mariuz commented Apr 14, 2026

Copy link
Copy Markdown
Contributor

@juarezr

Solution (recommended): Don’t run the container-based tests on Windows/macOS
Your action is a Docker container action (or otherwise requires Linux containers). Running it on Windows/macOS runners is brittle and (on GitHub-hosted Windows) often impossible without heavy custom provisioning.

Update the matrix to run Firebird container tests only on Ubuntu

ps: I tried :) in a separate pull request and failed to add Windows and macOS to the matrix

@juarezr

juarezr commented Apr 15, 2026

Copy link
Copy Markdown
Owner Author

@juarezr

Solution (recommended): Don’t run the container-based tests on Windows/macOS
Your action is a Docker container action (or otherwise requires Linux containers). Running it on Windows/macOS runners is brittle and (on GitHub-hosted Windows) often impossible without heavy custom provisioning.

I've stumbled on the same issues.
I believe this won't be trivial to do.

ps: I tried :) in a separate pull request and failed to add Windows and macOS to the matrix

I've noticed your efforts.
I still believe that this would be possible, but it will require some kind of condition or some restrictions on the architecture and OS versions.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants