Community for the Daft project and all things distributed data

Distributed Data Community

<https://github.com/Eventual-Inc/Daft/blob/main/.pre-commit-config.yaml#L78-L103>

is there a particular reason clippy _and_ check are run? <@U041QSEF2H2>

We want to ensure that the code compiles for both the python feature on and off.

Where-as we only care about cargo clippy not throwing any warnings in the case of all features on

image.png

just from

```complexity = { level = "deny", priority = -1 }
nursery = { level = "deny", priority = -1 }
perf = { level = "deny", priority = -1 }
style = { level = "deny", priority = -1 }
suspicious = { level = "deny", priority = -1 }
too_long_first_doc_paragraph = "allow"
derive_partial_eq_without_eq = "allow"
enum_variant_names = "allow"
module_name_repetitions = "allow"
option_if_let_else = "allow"
unsafe_block_in_unsafe_fn = "allow" # some pyo3 macros trigger this```


but my guess is compiler (hopefully) takes those out

<@U07P25T9PDW> That seems like a good idea to implement. Want to make some PRs that incrementally turn on these rules?

ok yea I will look into this. Some should be able to be automatically fixed as well so will look into that

<https://github.com/Eventual-Inc/Daft/pull/2875>