-
Notifications
You must be signed in to change notification settings - Fork 1.7k
Handle private named parameters in field formal conversion assists #62438
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?
Handle private named parameters in field formal conversion assists #62438
Conversation
Fixes dart-lang#62436 Updates convert_field_formal_to_normal.dart and convert_to_normal_parameter.dart to properly handle private named parameters by using the public name for the parameter declaration while preserving the private field name in the initializer.
|
Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). View this failed invocation of the CLA check for more information. For the most up to date status, view the checks section at the bottom of the pull request. |
|
@pashaarshad you need to sign CLA, or we can't review the code. |
|
@mraleph Thanks for the reminder. I’ll sign the Google CLA shortly. |
|
@pashaarshad you can read and sign CLA here: https://cla.developers.google.com/ |
|
Thank you for your contribution! This project uses Gerrit for code reviews. Your pull request has automatically been converted into a code review at: https://dart-review.googlesource.com/c/sdk/+/475120 Please wait for a developer to review your code review at the above link; you can speed up the review if you sign into Gerrit and manually add a reviewer that has recently worked on the relevant code. See CONTRIBUTING.md to learn how to upload changes to Gerrit directly. Additional commits pushed to this PR will update both the PR and the corresponding Gerrit CL. After the review is complete on the CL, your reviewer will merge the CL (automatically closing this PR). |
|
https://dart-review.googlesource.com/c/sdk/+/475120 has been updated with the latest commits from this pull request. |
|
@mraleph sir , now i have done the CLA ..................Now check the PR |
|
I noticed that #62436 is already closed as fixed. I think @munificent has already landed necessary changes? |
|
Sorry, @pashaarshad, I wasn't aware you were looking at this. My change handles @pashaarshad, do you want to rebase and remove your changes to Then ideally @bwilkerson or @scheglov would review it. Thank you for contributing to Dart! |
Fixes #62436
Updates
convert_field_formal_to_normal.dartandconvert_to_normal_parameter.dartto properly handle private named parameters.When converting
this._footo a normal parameter:foo)_foo = foo)private_named_parametersfeature is enabled