git: Use absolute path argument for commit_and_push()

This avoids the duplicate `relative_index_file()` calls in the background tasks

Author: Turbo87

src/git.rs

@@ -244,7 +244,8 @@ impl Repository {
     pub fn commit_and_push(&self, message: &str, modified_file: &Path) -> Result<(), PerformError> {
         println!("Committing and pushing \"{}\"", message);
 
-        self.perform_commit_and_push(message, modified_file)
+        let relative_path = modified_file.strip_prefix(self.checkout_path.path())?;
+        self.perform_commit_and_push(message, relative_path)
             .map(|_| println!("Commit and push finished for \"{}\"", message))
             .map_err(|err| {
                 eprintln!("Commit and push for \"{}\" errored: {}", message, err);

src/worker/git.rs

@@ -1,5 +1,5 @@
 use crate::background_jobs::Environment;
-use crate::git::{Crate, Credentials, Repository};
+use crate::git::{Crate, Credentials};
 use crate::models::Version;
 use chrono::Utc;
 use std::fs::{self, OpenOptions};
@@ -21,7 +21,7 @@ pub fn add_crate(env: &Environment, krate: Crate) -> Result<(), PerformError> {
 
     let message: String = format!("Updating crate `{}#{}`", krate.name, krate.vers);
 
-    repo.commit_and_push(&message, &Repository::relative_index_file(&krate.name))
+    repo.commit_and_push(&message, &dst)
 }
 
 /// Yanks or unyanks a crate version. This requires finding the index
@@ -61,7 +61,7 @@ pub fn yank(
         version.num
     );
 
-    repo.commit_and_push(&message, &Repository::relative_index_file(&krate))?;
+    repo.commit_and_push(&message, &dst)?;
 
     Ok(())
 }