Skip to content

[embedded] Update UserManual.md and EmbeddedSwiftStatus.md with some additional information #71523

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
Feb 13, 2024

Conversation

kubamracek
Copy link
Contributor

See the diff for details.

@kubamracek kubamracek added the embedded Embedded Swift label Feb 9, 2024
@kubamracek
Copy link
Contributor Author

@swift-ci please smoke test


```swift
func sayHello() {
#if hasFeature(Embedded)
Copy link
Member

Choose a reason for hiding this comment

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

$Embedded; hasFeature doesn't check if the feature is enabled if I recall correctly

Copy link
Contributor

Choose a reason for hiding this comment

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

This is an important distinction we should call out because user guidance is to use hasFeature(), not $.

Or maybe there should be a targetEnvironment(embedded) condition.

@@ -10,8 +10,8 @@ For an introduction and motivation into Embedded Swift, please see "[A Vision fo

This status table describes which of the following standard library features can be used in Embedded Swift:

| **Swift Standard Library Feature** | **Currently Supported In Embedded Swift** |
|---------------------------------------------|-----------------------------------------------------|
| **Swift Standard Library Feature** | **Currently Supported In Embedded Swift?** |
Copy link
Contributor

Choose a reason for hiding this comment

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

Fun fact, you don't need to pad the columns out. Markdown will render this nicely:

| **Foo** | **Bar** |
|-|-|
| Alpha. | Beta. |
| Gamma delta epsilon eridani. | Live long and prosper. |
Foo Bar
Alpha. Beta.
Gamma delta epsilon eridani. Live long and prosper.

@kubamracek kubamracek merged commit 6657349 into swiftlang:main Feb 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
embedded Embedded Swift
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants