Improved Swift 6 handling (#11)

* update timestamp handling

* resolve warning

* update devcontainer for swift 6

* add swift 6 mode compat

* test swift 5.9 and 6

* remove gha verbose logging

Author: jagreenwood

.devcontainer/devcontainer.json

@@ -1,12 +1,12 @@
 {
     "name": "Swift",
-    "image": "swift:5.9.1",
+    "image": "swift:6.0",
     "features": {
         "ghcr.io/devcontainers/features/common-utils:2": {
             "installZsh": "false",
             "username": "vscode",
-            "userUid": "1000",
-            "userGid": "1000",
+            "userUid": "1001",
+            "userGid": "1001",
             "upgradePackages": "false"
         },
         "ghcr.io/devcontainers/features/git:1": {

.github/workflows/swift.yml renamed to .github/workflows/swift-5.9.yml

@@ -1,4 +1,4 @@
-name: Unit Test
+name: Test Swift 5.9
 
 on:
   push:
@@ -8,11 +8,11 @@ on:
 
 jobs:
   build:
-
+    container: swift:5.9
     runs-on: ubuntu-latest
     steps:
     - uses: actions/checkout@v4
     - name: Build
-      run: swift build -v
+      run: swift build
     - name: Run tests
-      run: swift test -v
+      run: swift test

.github/workflows/swift-6.0.yml

@@ -0,0 +1,18 @@
+name: Test Swift 6.0
+
+on:
+  push:
+    branches: [ main ]
+  pull_request:
+    branches: [ main ]
+
+jobs:
+  build:
+    container: swift:6.0
+    runs-on: ubuntu-latest
+    steps:
+    - uses: actions/checkout@v4
+    - name: Build
+      run: swift build
+    - name: Run tests
+      run: swift test

Package.swift

@@ -25,7 +25,8 @@ let package = Package(
         .testTarget(
             name: "DataDogLogTests",
             dependencies: ["DataDogLog"]),
-    ]
+    ],
+    swiftLanguageVersions: [.version("6"), .v5]
 )
 
 #if os(Linux)

Sources/DataDogLog/Internal/Extensions/String+Timestamp.swift

@@ -4,8 +4,11 @@ extension String {
     static var timestamp: String {
         var buffer = [Int8](repeating: 0, count: 255)
         var timestamp = time(nil)
-        let localTime = localtime(&timestamp)
+        guard let localTime = localtime(&timestamp) else {
+            return "<unknown>"
+        }
         strftime(&buffer, buffer.count, "%Y-%m-%dT%H:%M:%S%z", localTime)
+
         return buffer.withUnsafeBufferPointer {
             $0.withMemoryRebound(to: CChar.self) {
                 String(cString: $0.baseAddress!)

Sources/DataDogLog/Internal/NetworkClient.swift

@@ -24,7 +24,7 @@ final class NetworkClient: Sendable {
 
     deinit {
 #if os(Linux)
-        client.shutdown()
+        client.shutdown { _ in }
 #endif
     }