-
Notifications
You must be signed in to change notification settings - Fork 10.5k
Emit fix-its for Return from initializer
missing variables
#80945
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
base: main
Are you sure you want to change the base?
Conversation
c7f786a
to
956adb6
Compare
Hi @eeckstein would love to have your input here please🙏🏻 |
I'm probably not the best person to review this. Maybe @rintaro can take a look? |
Return from initializer
Return from initializer
missing variables
@eeckstein I updated the example; I had accidentally pasted the output into the input earlier 🫠 additionally, i updated the PR with a video showing the change from Xcode, a little bit easier when visualized, the fix-it is what this PR is trying to add |
@rintaro would love to have your input on my PR please 🙏🏻 |
Apologies for not replying sooner. But before that, please add some test cases. You could update |
Avoid string dances
34b9b52
to
87253b8
Compare
tests updated with the new expected fix-its. |
ping 🙏🏻 |
@eeckstein Hi, can you suggest any reviewer for the PR ? 🙏🏼 |
Resolves #80944
11223.mov
This's my proposed solution for the issue.
the proposed solution:
init(....)
functioninit
to add,foo: Bar
self.foo = foo after {
which is the opening of brace of init functiontake the as input:
the following fix-its are emitted:
when applied, it will result to:
i opened the PR to discuss the solution