Copilot commented on code in PR #50401:
URL: https://github.com/apache/arrow/pull/50401#discussion_r3543995556
##########
c_glib/test/flight/test-client.rb:
##########
@@ -22,7 +22,7 @@ def setup
@server = nil
omit("Arrow Flight is required") unless defined?(ArrowFlight)
omit("Unstable on Windows") if Gem.win_platform?
- omit("Unstable on x86_64 macOS") if /x86_64-darwin/.match?(RUBY_PLATFORM)
+ omit("Unstable on macOS") if /darwin/.match?(RUBY_PLATFORM)
Review Comment:
This change skips Flight client tests on all macOS by matching /darwin/.
Previously this suite only skipped x86_64 macOS, so this now drops coverage for
arm64/macOS too. If the instability is specific to GitHub Actions (as described
in GH-50302), consider narrowing the skip to that environment (and keep the
existing x86_64 skip separate) so local macOS runs and/or non-GHA macOS runners
still exercise these tests. Apply the same narrowing in the other updated test
files that now omit on /darwin/.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
