Commit a9707bb

authored

Only apply standard swift settings on valid targets (#48)

Only apply standard swift settings on valid targets. The current check ignores plugins but that is not comprehensive enough.

1 parent 93b188b commit a9707bbCopy full SHA for a9707bb

File tree

-1

lines changed

-1

lines changed

Lines changed: 4 additions & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -54,11 +54,14 @@ let package = Package(`
`54`	`54`
`55`	`55`	`// --- STANDARD CROSS-REPO SETTINGS DO NOT EDIT --- //`
`56`	`56`	`for target in package.targets {`
`57`		`- if target.type != .plugin {`
	`57`	`+ switch target.type {`
	`58`	`+ case .regular, .test, .executable:`
`58`	`59`	`var settings = target.swiftSettings ?? []`
`59`	`60`	`// https://github.com/swiftlang/swift-evolution/blob/main/proposals/0444-member-import-visibility.md`
`60`	`61`	`settings.append(.enableUpcomingFeature("MemberImportVisibility"))`
`61`	`62`	`target.swiftSettings = settings`
	`63`	`+ case .macro, .plugin, .system, .binary: () // not applicable`
	`64`	`+ @unknown default: () // we don't know what to do here, do nothing`
`62`	`65`	`}`
`63`	`66`	`}`
`64`	`67`	`// --- END: STANDARD CROSS-REPO SETTINGS DO NOT EDIT --- //`

Comments

(0)