Skip to content

Commit

Permalink
remove redundant FIXME
Browse files Browse the repository at this point in the history
Signed-off-by: Tarek <[email protected]>
  • Loading branch information
tareknaser committed Jun 1, 2024
1 parent 8bd0480 commit 4296f73
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 4 deletions.
2 changes: 0 additions & 2 deletions ark-cli/src/commands/backup.rs
Original file line number Diff line number Diff line change
Expand Up @@ -15,8 +15,6 @@ pub struct Backup {
}

impl Backup {
// FIXME: The logic for backup should be moved out of `ark-cli`
// so that it can be used by other applications
pub fn run(&self) -> Result<(), AppError> {
let timestamp = timestamp().as_secs();
let backup_dir = home_dir()
Expand Down
3 changes: 1 addition & 2 deletions ark-cli/src/commands/storage/list.rs
Original file line number Diff line number Diff line change
Expand Up @@ -12,8 +12,7 @@ pub struct List {
root_dir: Option<PathBuf>,
#[clap(help = "Storage name")]
storage: Option<String>,
// TODO: What is the purpose of this field? Add a help message to clarify.
#[clap(short, long, action = clap::ArgAction::SetTrue)]
#[clap(short, long, action = clap::ArgAction::SetTrue, help = "Whether or not to use atomatic versioning")]
versions: bool,
#[clap(short, long, value_enum, help = "Storage kind of the resource")]
kind: Option<StorageType>,
Expand Down

0 comments on commit 4296f73

Please sign in to comment.