chore: Small fixups in the CLI code. (#136)

This commit just makes some small fixes in the CLI code, nothing
impacting behavior at all.

Signed-off-by: Andrew Lilley Brinker <alilleybrinker@gmail.com>

omnibor/src/bin/omnibor/find.rs

@@ -16,6 +16,7 @@ use tokio::sync::mpsc::Sender;
 pub async fn run(tx: &Sender<PrinterCmd>, args: &FindArgs) -> Result<()> {
     let FindArgs { url, path, format } = args;
 
+    // TODO(alilleybrinker): Correctly handle possible future hash formats.
     let id = ArtifactId::<Sha256>::id_url(url.clone())?;
     let url = id.url();
 
@@ -24,7 +25,7 @@ pub async fn run(tx: &Sender<PrinterCmd>, args: &FindArgs) -> Result<()> {
     loop {
         match entries.next().await {
             None => break,
-            Some(Err(e)) => tx.send(PrinterCmd::Message(Msg::error(e, *format))).await?,
+            Some(Err(e)) => tx.send(PrinterCmd::error(e, *format)).await?,
             Some(Ok(entry)) => {
                 let path = &entry.path();
 

omnibor/src/bin/omnibor/main.rs

@@ -31,7 +31,7 @@ async fn main() -> ExitCode {
     exit_code
 }
 
-/// Select and run the chosen chosen.
+/// Select and run the chosen command.
 async fn run(tx: &Sender<PrinterCmd>, cmd: &Command) -> Result<()> {
     match cmd {
         Command::Id(ref args) => id::run(tx, args).await,

omnibor/src/bin/omnibor/print.rs

@@ -87,7 +87,10 @@ impl Printer {
 /// A print queue message, either an actual message or a signals to end printing.
 #[derive(Debug, Clone)]
 pub enum PrinterCmd {
+    /// Shut down the printer task.
     End,
+
+    /// Print the following message.
     Message(Msg),
 }