Skip to content

Deduplicate package Swift settings#196

Open
fhwvtqdc2q-svg wants to merge 1 commit into
apple:mainfrom
fhwvtqdc2q-svg:codex/deduplicate-swift-settings
Open

Deduplicate package Swift settings#196
fhwvtqdc2q-svg wants to merge 1 commit into
apple:mainfrom
fhwvtqdc2q-svg:codex/deduplicate-swift-settings

Conversation

@fhwvtqdc2q-svg
Copy link
Copy Markdown

Summary

  • remove the initial per-target swiftSettings assignment from Package.swift
  • rely on the existing package target loop to apply ExistentialAny, MemberImportVisibility, and InternalImportsByDefault once to every target

Validation

  • ran git diff --check
  • verified Package.swift has a single ExistentialAny setting application path

Note: I could not run SwiftPM tests in this environment because Swift/Xcode is not installed on the Windows host.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant