Skip to content

Commit

Permalink
Merge pull request #123 from scipopt/allow-too-many-args
Browse files Browse the repository at this point in the history
Clippy: allow too many arguments
  • Loading branch information
mmghannam authored Jan 15, 2024
2 parents 0196aeb + d9b952e commit a54eec4
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 4 deletions.
3 changes: 3 additions & 0 deletions Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -17,3 +17,6 @@ doc-comment = "0.3.3"

[dev-dependencies]
rayon = "1.5.1"

[lints.clippy]
too_many_arguments = "allow"
2 changes: 0 additions & 2 deletions src/model.rs
Original file line number Diff line number Diff line change
Expand Up @@ -242,7 +242,6 @@ impl Model<ProblemCreated> {
/// # Returns
///
/// This function returns the `Model` instance for which the heuristic was included. This allows for method chaining.
#[allow(clippy::too_many_arguments)]
pub fn include_heur(
self,
name: &str,
Expand Down Expand Up @@ -574,7 +573,6 @@ pub trait ProblemOrSolving {
/// # Panics
///
/// This method panics if the constraint cannot be created in the current state.
#[allow(clippy::too_many_arguments)]
fn add_cons_quadratic(
&mut self,
lin_vars: Vec<Rc<Variable>>,
Expand Down
2 changes: 0 additions & 2 deletions src/scip.rs
Original file line number Diff line number Diff line change
Expand Up @@ -348,7 +348,6 @@ impl ScipPtr {
Ok(Constraint { raw: scip_cons })
}

#[allow(clippy::too_many_arguments)]
pub(crate) fn create_cons_quadratic(
&mut self,
lin_vars: Vec<Rc<Variable>>,
Expand Down Expand Up @@ -811,7 +810,6 @@ impl ScipPtr {
Ok(())
}

#[allow(clippy::too_many_arguments)]
pub(crate) fn include_heur(
&self,
name: &str,
Expand Down

0 comments on commit a54eec4

Please sign in to comment.