Skip to content

Remove --enable-experimental-swift-testing from the Getting Started doc. #181

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jan 12, 2024

Conversation

grynspan
Copy link
Contributor

@grynspan grynspan commented Jan 12, 2024

As of the January 11 daily toolchain, SwiftPM detects if a package includes swift-testing as a dependency and, if so, does not require a developer to pass --enable-experimental-swift-testing. This PR updates the package's Getting Started document to reflect that.

Checklist:

  • Code and documentation should follow the style of the Style Guide.
  • If public symbols are renamed or modified, DocC references should be updated.

… doc.

As of the January 11 daily toolchain, SwiftPM detects if a package includes swift-testing as a dependency and, if so, does not require a developer to pass `--enable-experimental-swift-testing`. This PR updates the package's Getting Started document to reflect that.
@grynspan grynspan added documentation 📚 Improvements or additions to documentation swiftpm-integration 📦 Swift Package Manager integration labels Jan 12, 2024
@grynspan grynspan self-assigned this Jan 12, 2024
@grynspan
Copy link
Contributor Author

@swift-ci please test

@grynspan
Copy link
Contributor Author

Failure is a known issue with SwiftPM. Tests all passed in CI. Merging.

@grynspan grynspan merged commit b6254bc into main Jan 12, 2024
@grynspan grynspan deleted the jgrynspan/enable-no-longer-required branch January 12, 2024 21:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation 📚 Improvements or additions to documentation swiftpm-integration 📦 Swift Package Manager integration
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants