Skip to content

Update SE-0110's status #832

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

Closed
wants to merge 2 commits into from
Closed

Update SE-0110's status #832

wants to merge 2 commits into from

Conversation

BasThomas
Copy link
Contributor

@benrimmington
Copy link
Contributor

@BasThomas, see the comments in PR 726.

Part of SE-0110 was reverted/deferred, due to a usability regression.

Which of the proposal states would be the most accurate?

@BasThomas
Copy link
Contributor Author

BasThomas commented Apr 14, 2018

I think "accepted with revisions" might make sense. Otherwise a "partially implemented" status (like you suggested) might be a good alternative. What do you think?

For what it's worth, this came to my attention because of this:

https://twitter.com/slava_pestov/status/983568033422622721

Horray for progress on the final implementation of SE-0110: swiftlang/swift#15843

@benrimmington
Copy link
Contributor

The proposed solution of SE-0110 is available in Swift 4.0.3, so Implemented (Swift 4) might be the best status.

The usability regression examples seem to be more about "tuple splatting" from SE-0029.

@masters3d
Copy link
Contributor

I think deferred is the best status given the supplied link. https://lists.swift.org/pipermail/swift-evolution-announce/2017-June/000386.html

@BasThomas
Copy link
Contributor Author

Ping @DougGregor — what would be the appropriate status?

@DougGregor
Copy link
Member

This proposal is mostly-but-not-completely implemented. I prefer to leave it in the “accepted” state until we are sure it is done, which we hope will be within the next few months

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants