proptest-rs / proptest

Hypothesis-like property testing for Rust
Apache License 2.0
1.69k stars 158 forks source link

proptest! macro generates clippy warnings #371

Closed SUPERCILEX closed 10 months ago

SUPERCILEX commented 1 year ago
warning: matching over `()` is more explicit
   --> vecs/src/freestanding_vec.rs:406:5
    |
406 | /     proptest! {
407 | |         #[test]
408 | |         #[cfg(not(miri))]
409 | |         fn all(path: Vec<Path>) {
...   |
445 | |         }
446 | |     }
    | |_____^
    |
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#ignored_unit_patterns
    = note: this warning originates in the macro `$crate::proptest_helper` which comes from the expansion of the macro `proptest` (in Nightly builds, run with -Z macro-backtrace for more info)
andrewbanchich commented 11 months ago

Looks like this was fixed in this commit. @matthew-russo would you be able to make a release with this fix sometime soon?

matthew-russo commented 11 months ago

i'll try to get a release out this week after we merge some pending PRs

andrewbanchich commented 11 months ago

Thanks!