Merge pull request #1269 from cloudflare/avery/remove-redundant-builds

removes redundant builds when watching

src/commands/dev/gcs/watch.rs

@@ -22,7 +22,6 @@ pub fn watch_for_changes(
     while let Ok(_) = receiver.recv() {
         let user = user.clone();
         let target = target.clone();
-        build(&target)?;
 
         // acquire the lock so incoming requests are halted
         // until the new script is ready for them

src/preview/mod.rs

@@ -181,9 +181,7 @@ fn watch_for_changes(
     let (tx, rx) = channel();
     watch_and_build(&target, Some(tx))?;
 
-    while let Ok(_e) = rx.recv() {
-        build(&target)?;
-
+    while let Ok(_) = rx.recv() {
         if let Ok(new_id) = upload(&mut target, user, sites_preview, verbose) {
             let script_id = format!("{}", new_id);
 

src/watch/mod.rs

@@ -46,7 +46,10 @@ pub fn watch_and_build(
                                 tx.send(()).expect("--watch change message failed to send");
                             }
                         }
-                        Err(_) => message::user_error("Something went wrong while watching."),
+                        Err(e) => {
+                            log::debug!("{:?}", e);
+                            message::user_error("Something went wrong while watching.")
+                        }
                     }
                 }
             });