Skip to content
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.

Sign up for GitHub

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

Jump to bottom

👷 Setup CI job matrix to run cargo test #17

Merged
merged 4 commits into from
Jul 8, 2024
Merged

👷 Setup CI job matrix to run cargo test #17

merged 4 commits into from
Jul 8, 2024
+29 −5

Commits on Jul 8, 2024

  1. 👷 Setup CI job matrix for cargo test 

    Ensure that Rust unit tests and doctests are checked too. Running on Ubuntu-24.04.
    @weiji14
    weiji14 committed Jul 8, 2024
    Configuration menu
    Copy the full SHA
    e9eafef View commit details
    Browse the repository at this point in the history

  2. 📌 Test on Rust stable, beta and nightly toolchains 

    Test on all three release channels of Rust.
    @weiji14
    weiji14 committed Jul 8, 2024
    Configuration menu
    Copy the full SHA
    8670050 View commit details
    Browse the repository at this point in the history

  3. 🚩 Add --crate-type=rlib compilation target 

    Needed to enable doctests, otherwise `cargo test --doc` would error with "warning: doc tests are not supported for crate type(s) `cdylib` in package `cog3pio`". Xref https://users.rust-lang.org/t/how-to-run-only-the-doctests/54048/2 and rust-lang/cargo#4717.
    @weiji14
    weiji14 committed Jul 8, 2024
    Configuration menu
    Copy the full SHA
    5677144 View commit details
    Browse the repository at this point in the history

  4. 💚 Fix broken doctest due to mismatched Array type 

    Update docstring in src/lib.rs to check for an Array3 output from the read_geotiff function. Patches bca4c18.
    @weiji14
    weiji14 committed Jul 8, 2024
    Configuration menu
    Copy the full SHA
    bd59c1c View commit details
    Browse the repository at this point in the history