Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

grynspan
Copy link
Contributor

@grynspan grynspan commented Nov 21, 2023

We now have functional support for swift-testing in Swift Package Manager (as of swiftlang/swift-package-manager#7047). We should update our documentation:

  • Point developers to the main branch instead of 5.10 again;
  • Show developers how to invoke swift test in a way that triggers swift-testing; and
  • Show developers how to use XCTestScaffold only when using an older toolchain (with the appropriate caveats and nuances emphasized.)

Resolves #126.

Checklist:

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

We now have functional support for swift-testing in Swift Package Manager (as of swiftlang/swift-package-manager#7047). We should update our documentation:

- Point developers to the main branch instead of 5.10 again;
- Show developers how to invoke `swift test` in a way that triggers swift-testing; and
- Show developers how to use `XCTestScaffold` _only_ when using an older toolchain (with the appropriate caveats and nuances emphasized.)

Resolves #126.
@grynspan grynspan self-assigned this Nov 21, 2023
@grynspan grynspan added documentation 📚 Improvements or additions to documentation swiftpm-integration 📦 Swift Package Manager integration labels Nov 21, 2023
Copy link
Contributor

@Kyle-Ye Kyle-Ye left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

// README.md
We are working to integrate it with the rest of the Swift ecosystem.

Maybe we can also update the description here to reflect the latest state directly on README.md

@grynspan
Copy link
Contributor Author

But we're not done working! :)

@Kyle-Ye
Copy link
Contributor

Kyle-Ye commented Nov 23, 2023

But we're not done working! :)

Got it. I thought the main part is to integrate with SwiftPM. Wondering what's else left.

@grynspan
Copy link
Contributor Author

I think that might be a better topic for the forums than for this PR! 😁

@grynspan grynspan force-pushed the jgrynspan/126-update-getting-started branch from 5a8dc8b to 1c616a0 Compare November 27, 2023 18:09
@grynspan grynspan marked this pull request as ready for review November 27, 2023 20:24
@grynspan
Copy link
Contributor Author

@swift-ci please test

Copy link
Contributor

@stmontgomery stmontgomery left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just one optional wording suggestion

@grynspan grynspan force-pushed the jgrynspan/126-update-getting-started branch from 1c616a0 to fb356ba Compare November 27, 2023 22:09
@grynspan
Copy link
Contributor Author

@swift-ci please test

@grynspan grynspan merged commit d38c66c into main Nov 27, 2023
@grynspan grynspan deleted the jgrynspan/126-update-getting-started branch November 27, 2023 22:16
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.

Update Getting Started docs to use Swift main branch features
4 participants