Skip to content

Commit

Permalink
Remove TODO
Browse files Browse the repository at this point in the history
I decided there's nothing to do about failing deserialization.
  • Loading branch information
lindhe committed Aug 10, 2023
1 parent 84c9d29 commit 0345b16
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions src/lib.rs
Original file line number Diff line number Diff line change
Expand Up @@ -30,8 +30,7 @@ fn get_config(file_path: &str) -> MetadataConfig {
Err(why) => panic!("couldn't open {}: {}", file_path, why),
Ok(file) => file,
};
serde_json::from_reader(config_file).expect("PLEASE HANDLE ME!") // TODO: Catch panics, e.g.
// for missing fields.
serde_json::from_reader(config_file).expect("Unable to deserialize JSON.")
}

/// Get the defalt value for vendor.
Expand Down

0 comments on commit 0345b16

Please sign in to comment.