Skip to content

Commit d54d891

Browse files
committed
ci: update clippy command to use --all-targets --all-features
- Updated CI configuration to use more comprehensive clippy checks - Fixed clippy warnings in test files: - Removed needless borrows in test_contexts.rs - Fixed function call borrows in setup.rs tests - All tests continue to pass
1 parent 0481a90 commit d54d891

3 files changed

Lines changed: 4 additions & 4 deletions

File tree

.github/workflows/ci.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -63,7 +63,7 @@ jobs:
6363
run: cargo fmt --all --check
6464

6565
- name: Run cargo clippy
66-
run: cargo clippy -- -D warnings
66+
run: cargo clippy --all-targets --all-features -- -D warnings
6767

6868
audit:
6969
name: Audit

tests/helpers/test_contexts.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -60,7 +60,7 @@ impl TestContextRepo {
6060
}
6161
#[cfg(windows)]
6262
{
63-
command.env("USERPROFILE", &self.home_dir.path());
63+
command.env("USERPROFILE", self.home_dir.path());
6464
}
6565

6666
command

tests/setup.rs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -128,7 +128,7 @@ Setup complete
128128
.to_string_lossy()
129129
)));
130130

131-
verify_prepare_commit_msg_global_hook(&ctx, &hooks_dir)
131+
verify_prepare_commit_msg_global_hook(&ctx, hooks_dir)
132132
}
133133

134134
#[test_context(TestContextRepo, skip_teardown)]
@@ -310,7 +310,7 @@ Setup complete
310310
.to_string_lossy()
311311
)));
312312

313-
verify_prepare_commit_msg_local_hook(&ctx, &hooks_dir)
313+
verify_prepare_commit_msg_local_hook(&ctx, hooks_dir)
314314
}
315315

316316
#[test_context(TestContextRepo, skip_teardown)]

0 commit comments

Comments
 (0)