Skip to content

[ChangeLog] Add an entry describing new behavior when passing type na… #37262

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
May 6, 2021

Conversation

xedin
Copy link
Contributor

@xedin xedin commented May 4, 2021

…me as an argument

@xedin xedin requested a review from slavapestov May 4, 2021 22:56
Copy link
Contributor

@slavapestov slavapestov left a comment

Choose a reason for hiding this comment

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

Thanks!

@xedin xedin force-pushed the changelog-for-61749633 branch from bb01c76 to a2063a8 Compare May 5, 2021 18:33
@xedin
Copy link
Contributor Author

xedin commented May 5, 2021

@swift-ci please smoke test

@xedin
Copy link
Contributor Author

xedin commented May 5, 2021

@swift-ci please smoke test Windows platform

@xedin
Copy link
Contributor Author

xedin commented May 5, 2021

@swift-ci please smoke test macOS platform

@xedin xedin merged commit 521a985 into swiftlang:main May 6, 2021
@@ -6,6 +6,24 @@ _**Note:** This is in reverse chronological order, so newer entries are added to
Swift 5.5
---------

* Type names are no longer allowed as an argument to a subscript parameter that expects a metatype type

```swift
Copy link
Collaborator

Choose a reason for hiding this comment

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

Sorry, @xedin, would you mind indenting these lines so that all of the text falls under the bullet point in the formatted document?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sure!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

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