statrs-dev / statrs

Statistical computation library for Rust
https://docs.rs/statrs/latest/statrs/
MIT License
582 stars 83 forks source link

review CI #207

Closed YeungOnion closed 3 months ago

YeungOnion commented 6 months ago

Since it's been some time, let us ensure that our CI is as useful as we need it to be.

A few ideas I think would be useful beyond running our tests,

FreezyLemon commented 5 months ago

I've recently set up CI workflows for some other Rust projects, so I could take a look at this if you're not working on it already.

Do you want a job running for all three major platforms (windows, macos, linux)? AFAICT, the project doesn't really have any platform-specific code. On the other hand, it's not that hard to set up..

YeungOnion commented 5 months ago

I think that'd be great, thanks!

One thing to mention: I'd like to also have a way to run the tests that rely on the NIST dataset we were using while still excluding the data from the repo (see #195)

YeungOnion commented 3 months ago
  • running clippy
  • generally fail on rustc warnings
  • dependabot
  • cross-platform

clippy, fail on warn, and cross-platform by @FreezyLemon in #215 dependabot by @YeungOnion in #238