Skip to content

Fix #57354: Fix-it for deprecated initializers removes the .init part from self.init #58545

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 2 commits into from
Jan 24, 2023

Conversation

cbjeukendrup
Copy link
Contributor

Now, the fix-it will only remove the .init part if the expr at the left side of the dot is a type, and not, for example, a self expr.

Resolves: #57354

@cbjeukendrup cbjeukendrup force-pushed the fixit_deprec_self_init branch from 7f33b3b to 9f1cf08 Compare May 2, 2022 11:35
@LucianoPAlmeida LucianoPAlmeida requested review from CodaFi and xedin May 2, 2022 18:23
@cbjeukendrup cbjeukendrup force-pushed the fixit_deprec_self_init branch from 9f1cf08 to d8546e8 Compare September 9, 2022 02:17
…nherit the message and renaming from the initializer of the superclass
@hborla
Copy link
Member

hborla commented Jan 20, 2023

@cbjeukendrup thank you for your contribution! @xymus and @tshortli know the most about the availability checker; I've tagged them to help review this change

@tshortli
Copy link
Contributor

@swift-ci please test

@tshortli
Copy link
Contributor

@swift-ci please test macOS

Copy link
Contributor

@tshortli tshortli left a comment

Choose a reason for hiding this comment

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

Looks good to me, thank you for this contribution!

@tshortli tshortli merged commit b53eba0 into swiftlang:main Jan 24, 2023
@cbjeukendrup cbjeukendrup deleted the fixit_deprec_self_init branch January 24, 2023 17:49
@cbjeukendrup
Copy link
Contributor Author

Thanks for reviewing, @tshortli!

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.

[SR-15026] Fix-it for deprecated initializers removes the .init part from self.init
4 participants