Skip to content

Commit

Permalink
Respond to comments.
Browse files Browse the repository at this point in the history
  • Loading branch information
nand4011 committed Aug 9, 2024
1 parent e8a66b2 commit e30bea2
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
4 changes: 2 additions & 2 deletions momento-cli-opts/src/lib.rs
Original file line number Diff line number Diff line change
Expand Up @@ -258,12 +258,12 @@ to help find opportunities for optimizations with Momento.
metric_collection_rate: u32,
#[arg(
long = "start-date",
help = "The UTC start date of the metric collection period. Will use (end-date - 30 days) if not provided. (YYYY-MM-DD)"
help = "The inclusive UTC start date of the metric collection period. Will use (end-date - 30 days) if not provided. (YYYY-MM-DD)"
)]
metric_start_date: Option<String>,
#[arg(
long = "end-date",
help = "The UTC end date of the metric collection period. Will use the current date if not provided. (YYYY-MM-DD)"
help = "The inclusive UTC end date of the metric collection period. Will use the current date if not provided. (YYYY-MM-DD)"
)]
metric_end_date: Option<String>,
},
Expand Down
2 changes: 1 addition & 1 deletion momento/src/commands/cloud_linter/linter_cli.rs
Original file line number Diff line number Diff line change
Expand Up @@ -293,7 +293,7 @@ fn get_metric_time_range(

fn parse_date_string(date: &str) -> Result<NaiveDateTime, CliError> {
let naive_date = NaiveDate::parse_from_str(date, "%Y-%m-%d").map_err(|_| CliError {
msg: "Date must be in ISO 8601 (YYYY-MM-DD) format".to_string(),
msg: "Date must be in YYYY-MM-DD format".to_string(),
})?;
naive_date.and_hms_opt(0, 0, 0).ok_or_else(|| CliError {
msg: "invalid time".to_string(),
Expand Down

0 comments on commit e30bea2

Please sign in to comment.