remove APIs deprecated on 0.4 (#62)

* remove APIs deprecated on 0.4

* accept test diff

Author: davidhewitt

src/exporters.rs

@@ -99,7 +99,7 @@ pub fn span_exporter(
 }
 
 /// Build a [`PushMetricExporter`] for passing to
-/// [`with_metrics_options()`][crate::LogfireConfigBuilder::with_metrics_options].
+/// [`with_metrics()`][crate::LogfireConfigBuilder::with_metrics].
 ///
 /// This uses `OTEL_EXPORTER_OTLP_PROTOCOL` and `OTEL_EXPORTER_OTLP_TRACES_PROTOCOL` environment
 /// variables to determine the protocol to use (or otherwise defaults to [`Protocol::HttpBinary`]).

src/lib.rs

@@ -190,39 +190,6 @@ pub enum ConfigureError {
     Other(#[from] Box<dyn std::error::Error + Send + Sync>),
 }
 
-#[expect(deprecated)]
-mod deprecated {
-    use std::str::FromStr;
-
-    /// Whether to print to the console.
-    #[derive(Debug, Clone, Copy, PartialEq, Eq, Hash, Default)]
-    #[cfg_attr(feature = "serde", derive(serde::Serialize, serde::Deserialize))]
-    #[cfg_attr(feature = "serde", serde(rename_all = "lowercase"))]
-    #[deprecated(since = "0.4.0", note = "use `ConsoleOptions` instead")]
-    pub enum ConsoleMode {
-        /// Write to console if no logfire token
-        #[default]
-        Fallback,
-        /// Force write to console
-        Force,
-    }
-
-    impl FromStr for ConsoleMode {
-        type Err = String;
-
-        fn from_str(s: &str) -> Result<Self, Self::Err> {
-            match s {
-                "fallback" => Ok(ConsoleMode::Fallback),
-                "force" => Ok(ConsoleMode::Force),
-                _ => Err(format!("invalid console mode: {s}")),
-            }
-        }
-    }
-}
-
-#[expect(deprecated)]
-use deprecated::ConsoleMode;
-
 /// Main entry point to configure logfire.
 ///
 /// This should be called once at the start of the program.
@@ -251,8 +218,6 @@ pub fn configure() -> LogfireConfigBuilder {
         send_to_logfire: None,
         token: None,
         console_options: Some(ConsoleOptions::default()),
-        #[expect(deprecated)]
-        console_mode: ConsoleMode::Force,
         additional_span_processors: Vec::new(),
         advanced: None,
         metrics: Some(MetricsOptions::default()),
@@ -271,10 +236,6 @@ pub struct LogfireConfigBuilder {
     // service_version: Option<String>,
     // environment: Option<String>,
     console_options: Option<ConsoleOptions>,
-    /// Deprecated setting, `Force` implies use `console_options`, `Fallback` will filter
-    /// them out if `send_to_logfire` is false.
-    #[expect(deprecated)]
-    console_mode: ConsoleMode,
 
     // config_dir: Option<PathBuf>,
     // data_dir: Option<Path>,
@@ -336,21 +297,6 @@ impl LogfireConfigBuilder {
         self
     }
 
-    /// Whether to log to the console.
-    #[must_use]
-    #[deprecated(since = "0.4.0", note = "use `with_console()` instead")]
-    #[expect(deprecated)]
-    pub fn console_mode(mut self, console_mode: ConsoleMode) -> Self {
-        // FIXME: remove this API and make it match Python, see `console_options()` below
-        match console_mode {
-            ConsoleMode::Fallback => {
-                self.console_options = None;
-            }
-            ConsoleMode::Force => self.console_options = Some(ConsoleOptions::default()),
-        }
-        self
-    }
-
     /// Sets console options. Set to `None` to disable console logging.
     ///
     /// If not set, will use `ConsoleOptions::default()`.
@@ -400,22 +346,6 @@ impl LogfireConfigBuilder {
         self
     }
 
-    /// Configure [metrics options](crate::config::MetricsOptions).
-    #[must_use]
-    #[deprecated(since = "0.4.0", note = "use `with_metrics` instead")]
-    pub fn with_metrics_options(mut self, metrics: MetricsOptions) -> Self {
-        self.metrics = Some(metrics);
-        self
-    }
-
-    /// Whether to enable metrics.
-    #[must_use]
-    #[deprecated(since = "0.4.0", note = "use `with_metrics(None)` to disable metrics")]
-    pub fn enable_metrics(mut self, enable: bool) -> Self {
-        self.enable_metrics = enable;
-        self
-    }
-
     /// Finish configuring Logfire.
     ///
     /// Because this configures global state for the opentelemetry SDK, this can typically only ever be called once per program.
@@ -537,13 +467,7 @@ impl LogfireConfigBuilder {
 
         let console_writer = self
             .console_options
-            // NB deprecated behaviour: if set to fallback and sending to logfire, disable console
-            .filter(
-                #[expect(deprecated)]
-                |_| !(self.console_mode == ConsoleMode::Fallback && send_to_logfire),
-            )
-            .map(ConsoleWriter::new)
-            .map(Arc::new);
+            .map(|o| Arc::new(ConsoleWriter::new(o)));
 
         if let Some(console_writer) = console_writer.clone() {
             tracer_provider_builder = tracer_provider_builder

tests/test_basic_exports.rs

@@ -1306,7 +1306,7 @@ fn test_basic_span() {
                         "code.lineno",
                     ),
                     value: I64(
-                        713,
+                        637,
                     ),
                 },
                 KeyValue {