Skip to content

Commit c16a172

Browse files
committed
Only apply standard swift settings on valid targets
1 parent ae88002 commit c16a172

File tree

1 file changed

+10
-24
lines changed

1 file changed

+10
-24
lines changed

Package.swift

Lines changed: 10 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -23,41 +23,27 @@ let swiftSettings: [SwiftSetting] = [
2323

2424
let package = Package(
2525
name: "swift-openapi-runtime",
26-
platforms: [
27-
.macOS(.v10_15), .macCatalyst(.v13), .iOS(.v13), .tvOS(.v13), .watchOS(.v6), .visionOS(.v1)
28-
],
29-
products: [
30-
.library(
31-
name: "OpenAPIRuntime",
32-
targets: ["OpenAPIRuntime"]
33-
)
34-
],
35-
dependencies: [
36-
.package(url: "https://github.com/apple/swift-http-types", from: "1.0.0"),
37-
],
26+
platforms: [.macOS(.v10_15), .macCatalyst(.v13), .iOS(.v13), .tvOS(.v13), .watchOS(.v6), .visionOS(.v1)],
27+
products: [.library(name: "OpenAPIRuntime", targets: ["OpenAPIRuntime"])],
28+
dependencies: [.package(url: "https://github.com/apple/swift-http-types", from: "1.0.0")],
3829
targets: [
3930
.target(
4031
name: "OpenAPIRuntime",
41-
dependencies: [
42-
.product(name: "HTTPTypes", package: "swift-http-types")
43-
],
44-
swiftSettings: swiftSettings
45-
),
46-
.testTarget(
47-
name: "OpenAPIRuntimeTests",
48-
dependencies: ["OpenAPIRuntime"],
32+
dependencies: [.product(name: "HTTPTypes", package: "swift-http-types")],
4933
swiftSettings: swiftSettings
50-
),
34+
), .testTarget(name: "OpenAPIRuntimeTests", dependencies: ["OpenAPIRuntime"], swiftSettings: swiftSettings),
5135
]
5236
)
5337

5438
// --- STANDARD CROSS-REPO SETTINGS DO NOT EDIT --- //
5539
for target in package.targets {
56-
if target.type != .plugin {
40+
switch target.type {
41+
case .regular, .test, .executable:
5742
var settings = target.swiftSettings ?? []
5843
// https://github.com/swiftlang/swift-evolution/blob/main/proposals/0444-member-import-visibility.md
5944
settings.append(.enableUpcomingFeature("MemberImportVisibility"))
6045
target.swiftSettings = settings
46+
case .macro, .plugin, .system, .binary: () // not applicable
47+
@unknown default: () // we don't know what to do here, do nothing
6148
}
62-
}
63-
// --- END: STANDARD CROSS-REPO SETTINGS DO NOT EDIT --- //
49+
}// --- END: STANDARD CROSS-REPO SETTINGS DO NOT EDIT --- //

0 commit comments

Comments
 (0)