Skip to content

Commit f42100d

Browse files
committed
commenting on dead code
1 parent 8bfd0a7 commit f42100d

File tree

4 files changed

+6
-2
lines changed

4 files changed

+6
-2
lines changed

Cargo.lock

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

Cargo.toml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
[package]
22
name = "codeowners"
3-
version = "0.2.4"
3+
version = "0.2.5"
44
edition = "2024"
55

66
[profile.release]

src/ownership/fast.rs

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -95,6 +95,8 @@ pub fn find_file_owners(project_root: &Path, config: &Config, file_path: &Path)
9595
}
9696
}
9797

98+
// TODO: remove this once we've verified the fast path is working
99+
// This is simply matching the order of behavior of the original codeowners CLI
98100
if file_owners.len() > 1 {
99101
file_owners.sort_by(|a, b| {
100102
let priority_a = a

src/runner.rs

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -262,6 +262,8 @@ impl Runner {
262262
self.validate()
263263
}
264264

265+
// TODO: remove this once we've verified the fast path is working
266+
#[allow(dead_code)]
265267
pub fn for_file(&self, file_path: &str) -> RunResult {
266268
let relative_file_path = Path::new(file_path)
267269
.strip_prefix(&self.run_config.project_root)

0 commit comments

Comments
 (0)