-
Notifications
You must be signed in to change notification settings - Fork 18
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Refactor Rust CI a bit #697
Comments
I remember reading somewhere that doc tests are really slow because each is compiled as its own binary. References: |
Yup, it's a bit of a tradeoff, but I still like having them generally because it's such a good way to ensure your docs Actually Work™. We'll save a lot by not running tests twice here: geoarrow-rs/.github/workflows/ci.yml Lines 58 to 59 in 77381c3
There may be some other sensible split-into-separate-jobs as well. |
With #696, the tests take a lot longer to run since there's more doc tests. We can probably break up the checks in
Lint and Test
to not run the entire test suite twice, sequentially.@gadomski can take this on after #696 lands.
The text was updated successfully, but these errors were encountered: