sq: Add some todos.

  - This is part of the effort of moving to clap3's derive API and
    profit from the added type safety.

2 files changed, 4 insertions, 0 deletions

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: