Skip to content

Commit

Permalink
Avoid mysterious Default::default()
Browse files Browse the repository at this point in the history
In `deploy::test::{low_level_simple, low_level_skip}`,
`Default::default()` was used to construct a `BTreeMap`, with the only
indication of the variable's type being the signature of
`RealActionRunner::new()`, which it was passed by reference to.

Clippy's default-trait-access lint caught this.
  • Loading branch information
Diomendius committed Apr 12, 2024
1 parent 7f942c1 commit 7b99955
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions src/deploy.rs
Original file line number Diff line number Diff line change
Expand Up @@ -697,7 +697,7 @@ mod test {

let opt = Options::default();
let handlebars = handlebars::Handlebars::new();
let variables = Default::default();
let variables = BTreeMap::new();

// Expectation:
// create_symlink
Expand Down Expand Up @@ -800,7 +800,7 @@ mod test {

let opt = Options::default();
let handlebars = handlebars::Handlebars::new();
let variables = Default::default();
let variables = BTreeMap::new();

// Expectation:
// create_symlink
Expand Down

0 comments on commit 7b99955

Please sign in to comment.