-
Notifications
You must be signed in to change notification settings - Fork 10.4k
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
[stdlib] Rename flatten() to joined() (SE-0133) #3809
Conversation
@swift-ci please smoke test and merge. |
No, there's a real failure (in addition to all the other stuff that's broken right now). Typo on my part. I'll fix it right now. |
Then let's try again, shall we? @swift-ci please smoke test and merge. |
I'm guessing it still won't pass due to all the UnsafePointer-related changes, but it's worth a shot :) |
@CodaFi How about another shot now that CI builds are working? |
@swift-ci please smoke test and merge. |
Blocked on #3816. Now that that's fixed this will take one more build to sort. |
Huh, now I know what the "update branch" button on github does. |
@swift-ci Please test |
LGTM. Linux failed for unrelated reasons, waiting for macOS. |
@jtbandes Just noticed one thing. There are still two references to |
@gribozavr Sure, I'll prepare a PR now. Also just submitted #3838 which is related. |
🎉 |
What's in this pull request?
SE-0133: rename
flatten()
tojoined()
.Remaining step (pending core team feedback) will be to add a default value to the
joined(separator:)
variant for collections of Strings.Before merging this pull request to apple/swift repository:
Triggering Swift CI
The swift-ci is triggered by writing a comment on this PR addressed to the GitHub user @swift-ci. Different tests will run depending on the specific comment that you use. The currently available comments are:
Smoke Testing
A smoke test on macOS does the following:
device standard libraries are not built.
version of these tests are not run.
A smoke test on Linux does the following:
tests are not run.
Validation Testing
Lint Testing
Note: Only members of the Apple organization can trigger swift-ci.