chore(main): use &Path instead of &PathBuf

Author: omarahm3

src/cli.rs

@@ -38,7 +38,7 @@ pub fn get_program_info() -> ProgramInfo {
         Err(_) => panic!("Can't get current working directory"),
     };
 
-    if args.len() == 0 {
+    if args.is_empty() {
         // TODO should probably show help here
         println!("Please enter a valid command");
         exit(1);

src/commands/projects.rs

@@ -17,7 +17,7 @@ impl AddProject {
 
         // TODO This is needed because if the command was passed with empty arguments then it will
         // somehow escape the None arm above
-        if args.len() == 0 {
+        if args.is_empty() {
             println!("You must specify at least the name of the project");
             exit(1);
         }
@@ -55,7 +55,7 @@ impl SyncProjectBranch {
             Some(args) => args,
         };
 
-        if args.len() == 0 {
+        if args.is_empty() {
             println!("You must specify project name and branch");
             exit(1);
         }

src/config.rs

@@ -3,6 +3,7 @@ use crate::git::git_current_branch;
 use serde::{Deserialize, Serialize};
 use std::fs;
 use std::io::prelude::*;
+use std::path::Path;
 use std::path::PathBuf;
 
 const CONFIG_INIT: &str = r#"
@@ -19,7 +20,7 @@ pub struct Project {
 }
 
 impl Project {
-    pub fn create(name: &str, path: &PathBuf) -> Project {
+    pub fn create(name: &str, path: &Path) -> Project {
         Project {
             name: name.to_string(),
             path: path.to_path_buf(),
@@ -136,7 +137,7 @@ fn read_config_file(path: PathBuf) -> Config {
     }
 }
 
-fn handle_config_file(path: &PathBuf) {
+fn handle_config_file(path: &Path) {
     // Get parent directory path
     let parent_dir = match path.parent() {
         None => panic!("Error getting parent directory from path"),
@@ -156,11 +157,11 @@ fn handle_config_file(path: &PathBuf) {
     }
 }
 
-fn path_exists(path: &PathBuf) -> bool {
+fn path_exists(path: &Path) -> bool {
     fs::metadata(path).is_ok()
 }
 
-fn create_config_file(path: &PathBuf) {
+fn create_config_file(path: &Path) {
     // Create the actual config file
     let mut file = match fs::File::create(path) {
         Err(err) => panic!(

src/git.rs

@@ -1,8 +1,9 @@
 use std::fs;
 use std::path::PathBuf;
+use std::path::Path;
 use std::process::Command;
 
-pub fn get_repositories(path: &PathBuf) -> Vec<PathBuf> {
+pub fn get_repositories(path: &Path) -> Vec<PathBuf> {
     let mut repositories: Vec<PathBuf> = Vec::new();
 
     // TODO Must make sure that projects paths are directory the moment user enters them