Apply standard Swift package configuration (#917)

Author: tinder-cfuller

.github/workflows/swift.yml

@@ -19,6 +19,6 @@ jobs:
     - name: Checkout source
       uses: actions/checkout@v6
     - name: Build
-      run: swift build -v
+      run: swift build -v -Xswiftc -warnings-as-errors
     - name: Test
       run: swift test -v

Makefile

@@ -61,12 +61,13 @@ docs:
 		-scheme "$(target)" \
 		-destination "$(destination)" \
 		-derivedDataPath "$(DERIVED_DATA_PATH)" \
-		-skipPackagePluginValidation
+		-skipPackagePluginValidation \
+		$(if $(filter $(open),OPEN),OTHER_DOCC_FLAGS="--warnings-as-errors",)
 	@find "$(DERIVED_DATA_PATH)" \
 		-type d \
 		-name "$(target).doccarchive" \
 		-exec cp -R {} "$(ARCHIVE_PATH)/" \;
-	$(if $(filter $(open),OPEN),@open "$(ARCHIVE_PATH)/$(target).doccarchive",)
+	@$(if $(filter $(open),OPEN),open "$(ARCHIVE_PATH)/$(target).doccarchive",:)
 
 .PHONY: site
 site: target ?= Nodes

Package.swift

@@ -123,17 +123,23 @@ let package = Package(
 
 package.targets.forEach { target in
 
-    // TODO: Remove upon enabling Swift 6 language mode:
+    let types: [Target.TargetType] = [
+        .regular,
+        .test,
+        .executable,
+    ]
+
+    guard types.contains(target.type)
+    else { return }
+
     target.swiftSettings = (target.swiftSettings ?? []) + [.enableExperimentalFeature("StrictConcurrency")]
 
-    if treatWarningsAsErrors {
-        target.swiftSettings = (target.swiftSettings ?? []) + [
-            // TODO: Remove unsafe flag upon upgrading to Swift tools v6.2 and uncomment subsequent settings:
-            .unsafeFlags(["-warnings-as-errors"]),
+//    if treatWarningsAsErrors {
+//        target.swiftSettings = (target.swiftSettings ?? []) + [
 //            .treatAllWarnings(as: .error),
 //            .treatWarning("DeprecatedDeclaration", as: .warning),
-        ]
-    }
+//        ]
+//    }
 
     if enableSwiftLintBuildToolPlugin {
         target.plugins = (target.plugins ?? []) + [