nuno-faria commented on PR #1566: URL: https://github.com/apache/datafusion-python/pull/1566#issuecomment-4622654572
> Oh... CI doesn't catch this because we run `pytest` on linux only. I wonder if there's reason enough to run it on windows/mac on merge to main? I don't think we need to run it on every PR commit because that would mean keeping more artifacts, but on merges to main we do get full release builds for all wheels. It would be nice to run them on main just in case. However, in my experience the errors are usually related to paths or some specific functionality required by the tests, so I think it's also not critical to run them. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
