Merge pull request #31 from ktoso/wip-logger-not-set

!logging #30 logger need not be { set }

Author: slashmo

Sources/BaggageLogging/LoggingBaggageContextCarrier.swift

@@ -29,5 +29,5 @@ public protocol LoggingBaggageContextCarrier: BaggageContextCarrier {
     ///
     /// Writes to the `logger` metadata SHOULD NOT be reflected in the `baggage`,
     /// however writes to the underlying `baggage` SHOULD be reflected in the `logger`.
-    var logger: Logger { get set }
+    var logger: Logger { get }
 }

Tests/BaggageLoggingTests/LoggingBaggageContextCarrierTests.swift

@@ -71,12 +71,7 @@ public struct ExampleFrameworkContext: LoggingBaggageContextCarrier {
 
     private var _logger: Logger
     public var logger: Logger {
-        get {
-            return self._logger.with(context: self.baggage)
-        }
-        set {
-            self._logger = newValue
-        }
+        return self._logger.with(context: self.baggage)
     }
 
     public init(context baggage: BaggageContext, logger: Logger) {
@@ -88,12 +83,7 @@ public struct ExampleFrameworkContext: LoggingBaggageContextCarrier {
 struct CoolFrameworkContext: LoggingBaggageContextCarrier {
     private var _logger: Logger = Logger(label: "some frameworks logger")
     var logger: Logger {
-        get {
-            return self._logger.with(context: self.baggage)
-        }
-        set {
-            self._logger = newValue
-        }
+        return self._logger.with(context: self.baggage)
     }
 
     var baggage: BaggageContext = .init()