chore: remove redundant cfg feature attributes

these modules are conditionally included by the same flags in lib
open-telemetry · Oct 20, 2024 · f95729d · f95729d
1 parent 3efb37d
commit f95729d
Show file tree

Hide file tree

Showing 3 changed files with 0 additions and 5 deletions.
diff --git a/opentelemetry-otlp/src/exporter/http/mod.rs b/opentelemetry-otlp/src/exporter/http/mod.rs
@@ -34,7 +34,6 @@ mod logs;
 mod trace;
 
 /// Configuration of the http transport
-#[cfg(any(feature = "http-proto", feature = "http-json"))]
 #[derive(Debug)]
 #[cfg_attr(
     all(
@@ -380,14 +379,12 @@ fn add_header_from_string(input: &str, headers: &mut HashMap<HeaderName, HeaderV
 }
 
 /// Expose interface for modifying builder config.
-#[cfg(any(feature = "http-proto", feature = "http-json"))]
 pub trait HasHttpConfig {
     /// Return a mutable reference to the config within the exporter builders.
     fn http_client_config(&mut self) -> &mut HttpConfig;
 }
 
 /// Expose interface for modifying builder config.
-#[cfg(any(feature = "http-proto", feature = "http-json"))]
 impl HasHttpConfig for HttpExporterBuilder {
     fn http_client_config(&mut self) -> &mut HttpConfig {
         &mut self.http_config

diff --git a/opentelemetry-otlp/src/exporter/tonic/mod.rs b/opentelemetry-otlp/src/exporter/tonic/mod.rs
@@ -496,7 +496,6 @@ mod tests {
     }
 
     #[test]
-    #[cfg(feature = "grpc-tonic")]
     fn test_convert_compression() {
         #[cfg(feature = "gzip-tonic")]
         assert!(tonic::codec::CompressionEncoding::try_from(Compression::Gzip).is_ok());

diff --git a/opentelemetry-otlp/src/metric.rs b/opentelemetry-otlp/src/metric.rs
@@ -67,7 +67,6 @@ impl<C> MetricsExporterBuilder<C> {
         }
     }
 
-    #[cfg(any(feature = "http-proto", feature = "http-json", feature = "grpc-tonic"))]
     pub fn with_temporality(self, temporality: Temporality) -> MetricsExporterBuilder<C> {
         MetricsExporterBuilder {
             client: self.client,