From 6d04926414a202f02be367811dec1e97b25ad9bd Mon Sep 17 00:00:00 2001 From: Nora Widdecke Date: Tue, 31 May 2022 10:20:15 +0200 Subject: sq: Add some todos. - This is part of the effort of moving to clap3's derive API and profit from the added type safety. --- sq/src/sq.rs | 3 +++ sq/src/sq_cli.rs | 1 + 2 files changed, 4 insertions(+) diff --git a/sq/src/sq.rs b/sq/src/sq.rs index 94ec6e11..a794f8a9 100644 --- a/sq/src/sq.rs +++ b/sq/src/sq.rs @@ -390,6 +390,8 @@ impl Config<'_> { } } +// TODO: Use `derive`d command structs. No more values_of +// TODO: Handling (and cli position) of global arguments fn main() -> Result<()> { let policy = &mut P::new(); @@ -569,6 +571,7 @@ fn main() -> Result<()> { &mut output, signatures, certs)?; }, + // TODO: Extract body to commands/armor.rs Some(("armor", m)) => { use clap::FromArgMatches; let command = sq_cli::ArmorCommand::from_arg_matches(m)?; diff --git a/sq/src/sq_cli.rs b/sq/src/sq_cli.rs index be01fc2e..9eddd7bc 100644 --- a/sq/src/sq_cli.rs +++ b/sq/src/sq_cli.rs @@ -8,6 +8,7 @@ pub fn build() -> Command<'static> { ) } +// TODO: use clap_derive for the whole CLI /// Defines the CLI. /// /// The order of top-level subcommands is: -- cgit v1.2.3