skipping codeowner file validation when generate-and-validate

Author: perryqh

src/main.rs

@@ -114,13 +114,13 @@ fn cli() -> Result<(), Error> {
     let ownership = Ownership::build(Project::build(&project_root, &codeowners_file_path, &config).change_context(Error::Io)?);
 
     match args.command {
-        Command::Validate => ownership.validate().change_context(Error::ValidationFailed)?,
+        Command::Validate => ownership.validate(false).change_context(Error::ValidationFailed)?,
         Command::Generate => {
             std::fs::write(codeowners_file_path, ownership.generate_file()).change_context(Error::Io)?;
         }
         Command::GenerateAndValidate => {
             std::fs::write(codeowners_file_path, ownership.generate_file()).change_context(Error::Io)?;
-            ownership.validate().change_context(Error::ValidationFailed)?
+            ownership.validate(true).change_context(Error::ValidationFailed)?
         }
         Command::ForFile { name } => {
             let file_owners = ownership.for_file(&name).change_context(Error::Io)?;

src/ownership.rs

@@ -86,15 +86,15 @@ impl Ownership {
     }
 
     #[instrument(level = "debug", skip_all)]
-    pub fn validate(&self) -> Result<(), ValidatorErrors> {
+    pub fn validate(&self, skip_codeowners_file_validation: bool) -> Result<(), ValidatorErrors> {
         info!("validating file ownership");
         let validator = Validator {
             project: self.project.clone(),
             mappers: self.mappers(),
             file_generator: FileGenerator { mappers: self.mappers() },
         };
 
-        validator.validate()
+        validator.validate(skip_codeowners_file_validation)
     }
 
     #[instrument(level = "debug", skip_all)]

src/ownership/validator.rs

@@ -36,7 +36,7 @@ pub struct Errors(Vec<Error>);
 
 impl Validator {
     #[instrument(level = "debug", skip_all)]
-    pub fn validate(&self) -> Result<(), Errors> {
+    pub fn validate(&self, skip_codeowners_file_validation: bool) -> Result<(), Errors> {
         let mut validation_errors = Vec::new();
 
         debug!("validate_invalid_team");
@@ -45,8 +45,10 @@ impl Validator {
         debug!("validate_file_ownership");
         validation_errors.append(&mut self.validate_file_ownership());
 
-        debug!("validate_codeowners_file");
-        validation_errors.append(&mut self.validate_codeowners_file());
+        if !skip_codeowners_file_validation {
+            debug!("validate_codeowners_file");
+            validation_errors.append(&mut self.validate_codeowners_file());
+        }
 
         if validation_errors.is_empty() {
             Ok(())