Move to SwiftLogger

Author: mikeger

Package.resolved

@@ -28,19 +28,16 @@ let targets: [PackageDescription.Target] = [
                            "TestConfigurator",
                            "Git",
                            "PathKit",
-                           "Rainbow",
                            "Yams",
                            .product(name: "ArgumentParser", package: "swift-argument-parser")]),
     .target(name: "DependencyCalculator",
-            dependencies: ["Workspace", "PathKit", "SelectiveTestLogger", "Git"]),
+            dependencies: ["Workspace", "PathKit", "Git", .product(name: "Logging", package: "swift-log")]),
     .target(name: "TestConfigurator",
-            dependencies: ["Workspace", "PathKit", "SelectiveTestLogger"]),
+            dependencies: ["Workspace", "PathKit", .product(name: "Logging", package: "swift-log")]),
     .target(name: "Workspace",
-            dependencies: ["XcodeProj", "SelectiveTestLogger"]),
+            dependencies: ["XcodeProj", .product(name: "Logging", package: "swift-log")]),
     .target(name: "Git",
-            dependencies: ["SelectiveTestShell", "SelectiveTestLogger", "PathKit"]),
-    .target(name: "SelectiveTestLogger",
-            dependencies: ["Rainbow"]),
+            dependencies: ["SelectiveTestShell", "PathKit", .product(name: "Logging", package: "swift-log")]),
     .target(name: "SelectiveTestShell"),
     .testTarget(
         name: "SelectiveTestingTests",
@@ -77,8 +74,8 @@ let package = Package(
         .package(url: "https://github.com/tuist/XcodeProj.git", .upToNextMajor(from: "9.0.2")),
         .package(url: "https://github.com/apple/swift-argument-parser.git", .upToNextMajor(from: "1.2.0")),
         .package(url: "https://github.com/kylef/PathKit.git", .upToNextMinor(from: "1.0.0")),
-        .package(url: "https://github.com/onevcat/Rainbow", .upToNextMajor(from: "4.0.0")),
         .package(url: "https://github.com/jpsim/Yams.git", from: "5.0.5"),
+        .package(url: "https://github.com/apple/swift-log", from: "1.6.0")
     ],
     targets: targets
 )

Package.swift

@@ -4,7 +4,7 @@
 
 import Foundation
 
-final class ThreadSafe<A> {
+final class ThreadSafe<A: Sendable>: @unchecked Sendable {
     private var _value: A
     private let queue = DispatchQueue(label: "ThreadSafe")
     init(_ value: A) {
@@ -22,8 +22,8 @@ final class ThreadSafe<A> {
     }
 }
 
-extension Array {
-    func concurrentMap<B>(_ transform: @escaping (Element) -> B) -> [B] {
+extension Array where Element: Sendable {
+    func concurrentMap<B: Sendable>(_ transform: @escaping @Sendable (Element) -> B) -> [B] {
         let result = ThreadSafe([B?](repeating: nil, count: count))
         DispatchQueue.concurrentPerform(iterations: count) { idx in
             let element = self[idx]

Sources/DependencyCalculator/ConcurrentMap.swift

@@ -4,7 +4,7 @@
 
 import Foundation
 import PathKit
-import SelectiveTestLogger
+import Logging
 import Workspace
 
 public extension WorkspaceInfo {
@@ -18,7 +18,7 @@ public extension WorkspaceInfo {
             } else if let targetFromFolder = targetForFolder(path) {
                 result.insert(targetFromFolder)
             } else {
-                Logger.message("Changed file at \(path) appears not to belong to any target")
+                logger.info("Changed file at \(path) appears not to belong to any target")
             }
         }
         if incldueIndirectlyAffected {

Sources/DependencyCalculator/DependencyCalculator.swift

@@ -4,16 +4,18 @@
 
 import Foundation
 import Git
-import PathKit
-import SelectiveTestLogger
+@preconcurrency import PathKit
+import Logging
 import SelectiveTestShell
 import Workspace
 import XcodeProj
 
+let logger = Logger(label: "cx.gera.XcodeSelectiveTesting")
+
 extension PBXBuildFile {
     func paths(projectFolder: Path) -> [Path] {
         guard let file else {
-            Logger.warning("PBXBuildFile without file: self=\(self), \n self.product=\(String(describing: product))")
+            logger.warning("PBXBuildFile without file: self=\(self), \n self.product=\(String(describing: product))")
             return []
         }
 
@@ -29,7 +31,7 @@ extension PBXBuildFile {
         }
 
         guard paths.count > 0 else {
-            Logger.warning("File without paths: self=\(self), \n self.file=\(String(describing: file)), \n self.product=\(String(describing: product))")
+            logger.warning("File without paths: self=\(self), \n self.file=\(String(describing: file)), \n self.product=\(String(describing: product))")
             return []
         }
 
@@ -176,12 +178,12 @@ extension WorkspaceInfo {
 
         for (targetName, dependOnTargets) in config.dependencies {
             guard let target = allTargets[targetName] else {
-                Logger.error("Config: Cannot resolve \(targetName) to any known target")
+                logger.error("Config: Cannot resolve \(targetName) to any known target")
                 continue
             }
             for dependOnTargetName in dependOnTargets {
                 guard let targetDependOn = allTargets[dependOnTargetName] else {
-                    Logger.error("Config: Cannot resolve \(dependOnTargetName) to any known target")
+                    logger.error("Config: Cannot resolve \(dependOnTargetName) to any known target")
                     continue
                 }
 
@@ -193,15 +195,15 @@ extension WorkspaceInfo {
 
         for (targetName, filesToAdd) in config.targetsFiles {
             guard let target = allTargets[targetName] else {
-                Logger.error("Config: Cannot resolve \(targetName) to any known target")
+                logger.error("Config: Cannot resolve \(targetName) to any known target")
                 continue
             }
 
             for filePath in filesToAdd {
                 let path = (basePath + filePath).absolute()
 
                 guard path.exists else {
-                    Logger.error("Config: Path \(path) does not exist")
+                    logger.error("Config: Path \(path) does not exist")
                     continue
                 }
 
@@ -272,7 +274,7 @@ extension WorkspaceInfo {
 
             for affectedByPath in metadata.affectedBy {
                 guard affectedByPath.exists else {
-                    Logger.warning("Path \(affectedByPath) is mentioned from package at \(metadata.path) but does not exist")
+                    logger.warning("Path \(affectedByPath) is mentioned from package at \(metadata.path) but does not exist")
                     continue
                 }
 
@@ -309,7 +311,7 @@ extension WorkspaceInfo {
             let absolutePath = path.parent() + localPackage.relativePath
 
             guard let newPackages = try? PackageTargetMetadata.parse(at: absolutePath) else {
-                Logger.warning("Cannot find local package at \(absolutePath)")
+                logger.warning("Cannot find local package at \(absolutePath)")
                 return
             }
             for package in newPackages {
@@ -323,15 +325,15 @@ extension WorkspaceInfo {
             // Target dependencies
             for dependency in target.dependencies {
                 guard let name = dependency.target?.name else {
-                    Logger.warning("Target without name: \(dependency)")
+                    logger.warning("Target without name: \(dependency)")
                     continue
                 }
 
                 if let dependencyTarget = targetsByName[name] {
                     dependsOn.insert(targetIdentity,
                                      dependOn: TargetIdentity.project(path: path, target: dependencyTarget))
                 } else {
-                    Logger.warning("Unknown target: \(name)")
+                    logger.warning("Unknown target: \(name)")
                     dependsOn.insert(targetIdentity,
                                      dependOn: TargetIdentity.project(path: path, targetName: name, testTarget: false))
                 }
@@ -341,7 +343,7 @@ extension WorkspaceInfo {
             for packageDependency in (target.packageProductDependencies ?? []) {
                 let package = packageDependency.productName
                 guard let packageMetadata = packagesByName[package] else {
-                    Logger.warning("Package \(package) not found")
+                    logger.warning("Package \(package) not found")
                     continue
                 }
                 dependsOn.insert(targetIdentity,

Sources/DependencyCalculator/DependencyGraph.swift

@@ -3,12 +3,12 @@
 //
 
 import Foundation
-import PathKit
-import SelectiveTestLogger
+@preconcurrency import PathKit
+import Logging
 import SelectiveTestShell
 import Workspace
 
-struct PackageTargetMetadata {
+struct PackageTargetMetadata: Sendable {
     let path: Path
     let affectedBy: Set<Path>
     let name: String

Sources/DependencyCalculator/PackageMetadata.swift

@@ -4,21 +4,23 @@
 
 import Foundation
 import PathKit
-import SelectiveTestLogger
+import Logging
 import SelectiveTestShell
 
+let logger = Logger(label: "cx.gera.XcodeSelectiveTesting")
+
 public extension Git {
     func changeset(baseBranch: String, verbose: Bool = false) throws -> Set<Path> {
         let gitRoot = try repoRoot()
 
         var currentBranch = try Shell.execOrFail("(cd \(gitRoot) && git branch --show-current)").trimmingCharacters(in: .newlines)
         if verbose {
-            Logger.message("Current branch: \(currentBranch)")
-            Logger.message("Base branch: \(baseBranch)")
+            logger.info("Current branch: \(currentBranch)")
+            logger.info("Base branch: \(baseBranch)")
         }
 
         if currentBranch.isEmpty {
-            Logger.warning("Missing current branch at \(path)")
+            logger.warning("Missing current branch at \(path)")
 
             currentBranch = "HEAD"
         }