CI should check more than just `--release`

Rust-GPU / rust-gpu

🐉 Making Rust a first-class language and ecosystem for GPU shaders 🚧

https://rust-gpu.github.io

Apache License 2.0

956 stars 27 forks source link

CI should check more than just `--release` #40

Closed LegNeato closed 2 weeks ago

LegNeato commented 1 month ago

Due to an underlying rustc change when forward porting, building without --release was broken on aarch64 (see https://github.com/Rust-GPU/rust-gpu/issues/29 and https://github.com/Rust-GPU/rust-gpu/issues/31). Unfortunately, CI only runs --release so we were blind to this breakage.

While running everything with and without --release is probably not worth the additional CI time, we should have at least one job on each platform that ensures both configs work at a basic level.

LegNeato commented 2 weeks ago

Fixed by https://github.com/Rust-GPU/rust-gpu/pull/52, thanks @zanciks ! 🥳