Remove rust-toolchain.toml file (#611)

Author: djc

.github/workflows/ci.yml

@@ -48,7 +48,7 @@ jobs:
       # the rust-toolchain.toml file as metrics-observer requires 1.74 to build. See
       # https://github.com/metrics-rs/metrics/pull/505#discussion_r1724092556 for more information.
       matrix:
-        rust_version: ['stable', 'nightly', '1.71.1']
+        rust_version: ['stable', 'nightly', '1.74.0', '1.71.1']
         include:
           - rust_version: '1.71.1'
             exclude-packages: '--exclude metrics-observer'

Cargo.toml

@@ -13,6 +13,7 @@ members = [
 
 [workspace.package]
 edition = "2021"
+rust-version = "1.71.1" # metrics-observer has a higher MSRV
 license = "MIT"
 repository = "https://github.com/metrics-rs/metrics"
 homepage = "https://github.com/metrics-rs/metrics"

metrics-benchmark/Cargo.toml

@@ -3,7 +3,7 @@ name = "metrics-benchmark"
 version = "0.1.1-alpha.5"
 authors = ["Toby Lawrence <toby@nuclearfurnace.com>"]
 edition = "2018"
-rust-version = "1.71.1"
+rust-version = { workspace = true }
 publish = false
 
 [dependencies]

metrics-benchmark/src/main.rs

@@ -100,7 +100,7 @@ impl Generator {
         Generator {
             t0: None,
             gauge: 0,
-            hist: HdrHistogram::<u64>::new_with_bounds(1, u64::max_value(), 3).unwrap(),
+            hist: HdrHistogram::<u64>::new_with_bounds(1, u64::MAX, 3).unwrap(),
             done,
             rate_counter,
         }
@@ -243,7 +243,7 @@ fn main() {
     let producers = matches.opt_str("producers").unwrap_or_else(|| "1".to_owned()).parse().unwrap();
     let mode = matches
         .opt_str("mode")
-        .map(|s| if s.to_ascii_lowercase() == "fast" { "fast" } else { "slow" })
+        .map(|s| if s.eq_ignore_ascii_case("fast") { "fast" } else { "slow" })
         .unwrap_or_else(|| "slow")
         .to_owned();
 
@@ -286,7 +286,7 @@ fn main() {
     let mut total = 0;
     let mut t0 = Instant::now();
 
-    let mut upkeep_hist = HdrHistogram::<u64>::new_with_bounds(1, u64::max_value(), 3).unwrap();
+    let mut upkeep_hist = HdrHistogram::<u64>::new_with_bounds(1, u64::MAX, 3).unwrap();
     for _ in 0..seconds {
         let t1 = Instant::now();
 

metrics-exporter-dogstatsd/Cargo.toml

@@ -2,7 +2,7 @@
 name = "metrics-exporter-dogstatsd"
 version = "0.9.6"
 edition = { workspace = true}
-rust-version = "1.71.1"
+rust-version = { workspace = true }
 
 description = "A metrics-compatible exporter for sending metrics to the Datadog Agent."
 license = { workspace = true }

metrics-exporter-dogstatsd/src/forwarder/sync.rs

@@ -102,7 +102,7 @@ impl ClientState {
 
                     return result;
                 }
-            };
+            }
         }
     }
 }
@@ -174,7 +174,7 @@ impl Forwarder {
             if u32::try_from(payloads.len()).is_err() {
                 error!(num_payloads = payloads.len(), "Too many payloads to send.");
                 continue;
-            };
+            }
 
             let splay_duration = next_flush.saturating_duration_since(Instant::now());
             debug!(

metrics-exporter-dogstatsd/src/storage.rs

@@ -184,7 +184,7 @@ pub(crate) enum Values<'a> {
     Sampled(Drain<'a>),
 }
 
-impl<'a> Iterator for Values<'a> {
+impl Iterator for Values<'_> {
     type Item = f64;
 
     fn next(&mut self) -> Option<Self::Item> {
@@ -195,7 +195,7 @@ impl<'a> Iterator for Values<'a> {
     }
 }
 
-impl<'a> ExactSizeIterator for Values<'a> {
+impl ExactSizeIterator for Values<'_> {
     fn len(&self) -> usize {
         match self {
             Values::Raw(values) => values.len(),

metrics-exporter-dogstatsd/src/writer.rs

@@ -519,15 +519,16 @@ impl PayloadWriter {
     }
 }
 
+#[allow(clippy::doc_link_with_quotes)]
 /// Iterator over all payloads written by a `PayloadWriter`.
 ///
-/// The source payload buffer is immediately drained of consumed data during the creation of this iterator (as known as
+/// The source payload buffer is immediately drained of consumed data during the creation of this iterator (also known as
 /// ["pre-pooping our pants"][everyone_poops]). This ensures that the end state - the payload buffer contains only
 /// preserved bytes (like length prefixes) - is established immediately.
 ///
 /// [everyone_poops]: https://faultlore.com/blah/everyone-poops/
 pub struct Payloads<'a> {
-    payloads_buf: Vec<u8>,
+    buf: Vec<u8>,
     start: usize,
     offsets: Drain<'a, usize>,
 }
@@ -539,7 +540,7 @@ impl<'a> Payloads<'a> {
         // for the next write operation.
         let drain_size = offsets.last().copied().unwrap_or(0);
         Self {
-            payloads_buf: payload_buf.drain(0..drain_size).collect(),
+            buf: payload_buf.drain(0..drain_size).collect(),
             start: 0,
             offsets: offsets.drain(..),
         }
@@ -556,7 +557,7 @@ impl<'a> Payloads<'a> {
     pub fn next_payload(&mut self) -> Option<&[u8]> {
         let offset = self.offsets.next()?;
 
-        let offset_buf = &self.payloads_buf[self.start..offset];
+        let offset_buf = &self.buf[self.start..offset];
         self.start = offset;
 
         Some(offset_buf)

metrics-exporter-prometheus/Cargo.toml

@@ -2,7 +2,7 @@
 name = "metrics-exporter-prometheus"
 version = "0.17.2"
 edition = "2018"
-rust-version = "1.71.1"
+rust-version = { workspace = true }
 
 description = "A metrics-compatible exporter for sending metrics to Prometheus."
 license = { workspace = true }

metrics-exporter-prometheus/src/distribution.rs

@@ -13,7 +13,7 @@ use metrics_util::{
 
 const DEFAULT_SUMMARY_BUCKET_COUNT: NonZeroU32 = match NonZeroU32::new(3) {
     Some(v) => v,
-    None => [][0],
+    None => unreachable!(),
 };
 const DEFAULT_SUMMARY_BUCKET_DURATION: Duration = Duration::from_secs(20);
 
@@ -123,7 +123,7 @@ impl DistributionBuilder {
     }
 
     /// Returns the distribution type for the given metric key.
-    pub fn get_distribution_type(&self, name: &str) -> &str {
+    pub fn get_distribution_type(&self, name: &str) -> &'static str {
         if self.buckets.is_some() {
             return "histogram";
         }