fix: remove unnecessary cfg macros

hstove · hstove · commit eef55e5ede53 · 2025-01-17T07:13:26.000-08:00
diff --git a/stacks-signer/src/monitoring/mod.rs b/stacks-signer/src/monitoring/mod.rs
@@ -50,13 +50,9 @@ pub mod actions {
     }
 
     /// Increment the block responses sent counter
-    #[allow(unused_variables)]
     pub fn increment_block_responses_sent(accepted: bool) {
-        #[cfg(feature = "monitoring_prom")]
-        {
-            let label_value = if accepted { "accepted" } else { "rejected" };
-            BLOCK_RESPONSES_SENT.with_label_values(&[label_value]).inc();
-        }
+        let label_value = if accepted { "accepted" } else { "rejected" };
+        BLOCK_RESPONSES_SENT.with_label_values(&[label_value]).inc();
     }
 
     /// Increment the number of block proposals received

Original file line number	Diff line number	Diff line change
`@@ -50,13 +50,9 @@ pub mod actions {`
`50`	`50`	`}`
`51`	`51`
`52`	`52`	`/// Increment the block responses sent counter`
`53`		`- #[allow(unused_variables)]`
`54`	`53`	`pub fn increment_block_responses_sent(accepted: bool) {`
`55`		`- #[cfg(feature = "monitoring_prom")]`
`56`		`- {`
`57`		`- let label_value = if accepted { "accepted" } else { "rejected" };`
`58`		`- BLOCK_RESPONSES_SENT.with_label_values(&[label_value]).inc();`
`59`		`- }`
	`54`	`+ let label_value = if accepted { "accepted" } else { "rejected" };`
	`55`	`+ BLOCK_RESPONSES_SENT.with_label_values(&[label_value]).inc();`
`60`	`56`	`}`
`61`	`57`
`62`	`58`	`/// Increment the number of block proposals received`