cc-rs icon indicating copy to clipboard operation
cc-rs copied to clipboard

Cut down on CI steps?

Open madsmtm opened this issue 11 months ago • 2 comments

Since https://github.com/rust-lang/cc-rs/pull/1344 and https://github.com/rust-lang/cc-rs/pull/1384, we have a lot of CI steps.

We should maybe consider cutting down on it, by:

  • Only testing --release and --features parallel on a few select platforms?
  • Merging some tests that run on the same platform (related to https://github.com/rust-lang/cc-rs/issues/1390).

Because while they may run in parallel, they still consume both power, and from my understanding also money from the Rust org (?).

madsmtm avatar Feb 05 '25 09:02 madsmtm

The tests added in PR #1384 are important for preventing regressions and more tests along those lines are needed. In the last year or so there have been lots of regressions in cc-rs that could be prevented with such testing.

briansmith avatar Feb 12 '25 04:02 briansmith

I didn't mean to imply that we should cut down on test coverage; I only meant that I fear that the way our current CI runs may be overly wasteful, and that in turn discourages me from adding more tests.

madsmtm avatar Feb 14 '25 11:02 madsmtm