spacejam / sled

the champagne of beta embedded databases
Apache License 2.0
8.18k stars 383 forks source link

Apply `rustfmt` and `clippy` fixes across crate #1421

Closed clubby789 closed 2 years ago

clubby789 commented 2 years ago

Add clippy and formatting fixes to allow CI to pass (#1418).

clubby789 commented 2 years ago

~~This seems to cause a few tests image to fail locally, possibly the automatic clippy fixes broke the code.~~ Made some mistakes while applying suggestions, fixed now.

clubby789 commented 2 years ago

I also disabled the missing_const_for_fn lint as that has some issues with false positives (i.e. when an operation is only const with unstable features)

spacejam commented 2 years ago

this would cause too many conflicts with private branches. If there are specific correctness-related diffs, please feel free to open a separate PR with them.