From 4830a261807ea149760039897ec36ac95aaeadc5 Mon Sep 17 00:00:00 2001
From: "hinto.janai" <hinto.janai@protonmail.com>
Date: Wed, 19 Jun 2024 17:57:11 -0400
Subject: [PATCH] add TODO for `strum`

---
 Cargo.toml              | 2 +-
 rpc/types/src/status.rs | 6 +++++-
 2 files changed, 6 insertions(+), 2 deletions(-)

diff --git a/Cargo.toml b/Cargo.toml
index 2ec0359..daf4ef6 100644
--- a/Cargo.toml
+++ b/Cargo.toml
@@ -75,7 +75,7 @@ rayon                 = { version = "1.9.0", default-features = false }
 serde_bytes           = { version = "0.11.12", default-features = false }
 serde_json            = { version = "1.0.108", default-features = false }
 serde                 = { version = "1.0.190", default-features = false }
-strum                 = { version = "0.26.2", default-features = false }
+strum                 = { version = "0.26.2", default-features = false } # TODO(hinto): do we need this?
 thiserror             = { version = "1.0.50", default-features = false }
 thread_local		  = { version = "1.1.7", default-features = false }
 tokio-util            = { version = "0.7.10", default-features = false }
diff --git a/rpc/types/src/status.rs b/rpc/types/src/status.rs
index 9a4a316..fc19450 100644
--- a/rpc/types/src/status.rs
+++ b/rpc/types/src/status.rs
@@ -7,7 +7,11 @@ use strum::{
     FromRepr, IntoStaticStr, VariantNames,
 };
 
-//---------------------------------------------------------------------------------------------------- TODO
+// TODO(hinto):
+// Do we need `strum`? Are there other types
+// (maybe outside of this crate) that will use it?
+
+//---------------------------------------------------------------------------------------------------- Status
 /// RPC response status.
 ///
 /// This type represents `monerod`'s frequently appearing string field, `status`.