Improving project build speed (#44)

* improving project build speed

* initializing vectors with capacity

Author: perryqh

Cargo.lock

@@ -10,10 +10,12 @@ debug = true
 clap = { version = "4.5.20", features = ["derive"] }
 clap_derive = "4.5.18"
 error-stack = "0.5.0"
-glob-match = "0.2.1"
+fast-glob = "0.4.0"
 ignore = "0.4.23"
 itertools = "0.13.0"
 path-clean = "1.0.1"
+lazy_static = "1.5.0"
+num_cpus = "1.16.0"
 rayon = "1.10.0"
 regex = "1.11.1"
 serde = { version = "1.0.214", features = ["derive"] }

Cargo.toml

@@ -10,7 +10,7 @@ pub mod tests {
 
     use tempfile::tempdir;
 
-    use crate::{ownership::Ownership, project::Project};
+    use crate::{ownership::Ownership, project_builder::ProjectBuilder};
 
     macro_rules! ownership {
         ($($test_files:expr),+) => {{
@@ -110,13 +110,15 @@ pub mod tests {
         let config = serde_yaml::from_reader(config_file)?;
 
         let codeowners_file_path = &test_config.temp_dir_path.join(".github/CODEOWNERS");
-        let project = Project::build(&test_config.temp_dir_path, codeowners_file_path, &config)?;
+        let mut builder = ProjectBuilder::new(&config, test_config.temp_dir_path.clone(), codeowners_file_path.clone());
+        let project = builder.build()?;
         let ownership = Ownership::build(project);
         if test_config.generate_codeowners {
             std::fs::write(codeowners_file_path, ownership.generate_file())?;
         }
         // rebuild project to ensure new codeowners file is read
-        let project = Project::build(&test_config.temp_dir_path, codeowners_file_path, &config)?;
+        let mut builder = ProjectBuilder::new(&config, test_config.temp_dir_path.clone(), codeowners_file_path.clone());
+        let project = builder.build()?;
         Ok(Ownership::build(project))
     }
 

src/common_test.rs

@@ -1,6 +1,6 @@
 use ownership::{FileOwner, Ownership};
 
-use crate::project::Project;
+use crate::project_builder::ProjectBuilder;
 use clap::{Parser, Subcommand};
 use core::fmt;
 use error_stack::{Context, Result, ResultExt};
@@ -15,6 +15,7 @@ mod common_test;
 mod config;
 mod ownership;
 mod project;
+mod project_builder;
 
 #[derive(Subcommand, Debug)]
 enum Command {
@@ -114,7 +115,9 @@ fn cli() -> Result<(), Error> {
 
     let config = serde_yaml::from_reader(config_file).change_context(Error::Io)?;
 
-    let ownership = Ownership::build(Project::build(&project_root, &codeowners_file_path, &config).change_context(Error::Io)?);
+    let mut builder = ProjectBuilder::new(&config, project_root, codeowners_file_path.clone());
+    let project = builder.build().change_context(Error::Io)?;
+    let ownership = Ownership::build(project);
 
     match args.command {
         Command::Validate => ownership.validate().change_context(Error::ValidationFailed)?,

src/main.rs

@@ -1,4 +1,4 @@
-use glob_match::glob_match;
+use fast_glob::glob_match;
 use std::{
     collections::HashMap,
     fmt::{self, Display},