Skip to content

Conversation

stevapple
Copy link
Contributor

No description provided.

@benrimmington benrimmington requested a review from xedin December 9, 2021 05:46
@DougGregor
Copy link
Member

I don't believe this feature has been enabled by default everywhere for 5.6 yet. @xedin can comment and okay the merge when it is enabled.

@xedin
Copy link
Contributor

xedin commented Dec 10, 2021

Yeah, unfortunately the enabling had to be rolled back, I'm working through a couple of remaining issues and once re-landed, going to update this thread.

Copy link
Contributor

@xedin xedin left a comment

Choose a reason for hiding this comment

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

We can finally land this PR since all of the issues related to multi-statement closure inference have been addressed.

@stevapple stevapple requested a review from xedin March 15, 2022 23:26
Copy link
Contributor

@xedin xedin left a comment

Choose a reason for hiding this comment

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

Thank you!

@xedin xedin merged commit 0617cf3 into swiftlang:main Mar 16, 2022
@stevapple stevapple deleted the se-0326-status branch March 16, 2022 17:26
@xedin
Copy link
Contributor

xedin commented Mar 16, 2022

@stevapple Would you like to write a CHANGELOG entry for this feature?

@stevapple
Copy link
Contributor Author

@xedin Just opened swiftlang/swift#41852 for this👌

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.

3 participants