search

aldanor / hdf5-rust

HDF5 for Rust
https://docs.rs/hdf5
Apache License 2.0
310 stars 84 forks source link

Apply lints #238

Closed rytheo closed 1 year ago

rytheo commented 1 year ago

Fixes the latest clippy/fmt warnings.

mulimoen commented 1 year ago

I'm wondering if this conflicts with MSRV. Could you bump the CI run for this test to ubuntu 20.04?

rytheo commented 1 year ago

@mulimoen It does conflict. The current MSRV is 1.54; new format string syntax is 1.58 and let-else is 1.65.

rytheo commented 1 year ago

Do we need to bump the MSRV or change the linter to not suggest new syntax?