Skip to content

Conversation

@grynspan
Copy link
Contributor

Explanation: Changes package templates to use Swift Testing by default instead of XCTest and removes package dependencies on the swift-testing package.
Scope: swift package init, swift package add-target
Issue: N/A
Original PR: #7872
Risk: Moderate
Testing: Original PR tested with toolchain builds.
Reviewer: @bnbarham @briancroom @xedin

…rather than as a package dependency. (#7872)

This PR updates the package templates used by `swift package init` to
use Swift Testing by default where possible, and to not add a package
dependency on the Swift Testing open-source repository since it is now
building in the toolchain.

Resolves rdar://128272585.
@grynspan grynspan added enhancement swift package init Package templates command changes command-line interface swift package `swift package` subcommand labels Aug 23, 2024
@grynspan grynspan self-assigned this Aug 23, 2024
@grynspan grynspan requested a review from a team as a code owner August 23, 2024 18:04
@grynspan
Copy link
Contributor Author

@swift-ci test

`canImport(Testing)` currently fails on the full toolchain test jobs due
to various issues when building for both x86 and arm64. Temporarily
remove this check until we can fix the underlying problems there.
@grynspan
Copy link
Contributor Author

@swift-ci test

@grynspan
Copy link
Contributor Author

@swift-ci test Windows

@grynspan
Copy link
Contributor Author

@swift-ci test

@grynspan grynspan enabled auto-merge (squash) August 27, 2024 00:26
@grynspan grynspan disabled auto-merge August 27, 2024 00:26
@shahmishal shahmishal merged commit 5bd155f into release/6.0.0 Aug 27, 2024
@shahmishal shahmishal deleted the jgrynspan/update-package-templates-to-use-swift-testing-6.0.0 branch August 27, 2024 01:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

command-line interface enhancement swift package init Package templates command changes swift package `swift package` subcommand

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants