scipopt / russcip

Rust interface for SCIP
https://crates.io/crates/russcip
Apache License 2.0
30 stars 10 forks source link

Consider allowing `clippy::too_many_arguments` directly in `Cargo.toml` #122

Closed mohammadfawaz closed 5 months ago

mohammadfawaz commented 5 months ago

It's likely that many functions in russcip will require a large numer of arguments to match the actual SCIP C-API. Instead of having to add #[allow(clippy::too_many_arguments)] everywhere like I had to do in #121, it's probably better to simply add

[lints.clippy]
too_many_arguments = "allow"

in Cargo.toml

An alternative is to modify the limit on what "too many" means but that seems unnecessary.

mmghannam commented 5 months ago

Thank you for the great suggestion!