worker::git: Change version argument of yank() to String

We only need the `num` field, so there is no need for us to save the full `Version` in the task payload

Author: Turbo87

src/controllers/version/yank.rs

@@ -61,7 +61,7 @@ fn modify_yank(req: &mut dyn RequestExt, yanked: bool) -> EndpointResult {
 
     insert_version_owner_action(&conn, version.id, user.id, api_token_id, action)?;
 
-    worker::yank(krate.name, version, yanked).enqueue(&conn)?;
+    worker::yank(krate.name, version.num, yanked).enqueue(&conn)?;
 
     ok_true()
 }

src/worker/git.rs

@@ -1,6 +1,5 @@
 use crate::background_jobs::Environment;
 use crate::git::{Crate, Credentials};
-use crate::models::Version;
 use chrono::Utc;
 use std::fs::{self, OpenOptions};
 use std::io::prelude::*;
@@ -32,7 +31,7 @@ pub fn add_crate(env: &Environment, krate: Crate) -> Result<(), PerformError> {
 pub fn yank(
     env: &Environment,
     krate: String,
-    version: Version,
+    version_num: String,
     yanked: bool,
 ) -> Result<(), PerformError> {
     let repo = env.lock_index()?;
@@ -44,7 +43,7 @@ pub fn yank(
         .map(|line| {
             let mut git_crate = serde_json::from_str::<Crate>(line)
                 .map_err(|_| format!("couldn't decode: `{}`", line))?;
-            if git_crate.name != krate || git_crate.vers != version.num {
+            if git_crate.name != krate || git_crate.vers != version_num {
                 return Ok(line.to_string());
             }
             git_crate.yanked = Some(yanked);
@@ -58,7 +57,7 @@ pub fn yank(
         "{} crate `{}#{}`",
         if yanked { "Yanking" } else { "Unyanking" },
         krate,
-        version.num
+        version_num
     );
 
     repo.commit_and_push(&message, &dst)?;