message: Refactor: returns proper error instead of unwrapping during license copy
diff: @@ -15,9 +15,15 @@ fn glob_license_files(path: &Path) -> Result<Vec<String>, failure::Error> {
  for entry in glob(path.join("LICENSE*").to_str().unwrap())? {
  match entry {
  Ok(globed_path) => {
  - license_files.push(String::from(
  - globed_path.file_name().unwrap().to_str().unwrap(),
  - ));
  + let file_name = match globed_path.file_name() {
  + Some(file_name) => file_name,
  + None => return Err(format_err!("Could not get file name from path")),
  + };
  + let file_name_string = match file_name.to_str() {
  + Some(file_name_string) => file_name_string,
  + None => return Err(format_err!("Could not convert filename to string")),
  + };
  + license_files.push(String::from(file_name_string));
  }
  Err(e) => println!("{:?}", e),
  }
  @@ -50,14 +56,14 @@ pub fn copy_from_crate(
  match license_files {
  Ok(files) => {
  - if files.len() == 0 {
  + if files.is_empty() {
  PBAR.info("License key is set in Cargo.toml but no LICENSE file(s) were found; Please add the LICENSE file(s) to your project directory");
  return Ok(());
  }
  for license_file in files {
  let crate_license_path = path.join(&license_file);
  let new_license_path = out_dir.join(&license_file);
  - if let Err(_) = fs::copy(&crate_license_path, &new_license_path) {
  + if fs::copy(&crate_license_path, &new_license_path).is_err() {
  PBAR.info("origin crate has no LICENSE");
  }
  }
  