-
Notifications
You must be signed in to change notification settings - Fork 12.7k
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
Destructured parameter properties do not introduce properties into their respective class #1541
Comments
@jonathandturner and I spoke offline and decided that the binding parameters should introduce properties of the appropriate visibility. |
@ahejlsberg and I spoke offline and decided that binding parameters should not introduce properties of the appropriate visibility. |
Reopening to update the spec. |
This is Fixed as far as the work goes; the spec work needs to be done. |
Let's create new bugs for spec work, that way you can close this with the fixed label. Otherwise we should remove the fixed label from this one, which then makes confusing the fact that it does represent dev work that fixed an issue. |
Agreed; I've opened up #1691 instead. |
which gives us
The question is should we be adding properties when destructuring in the face of a visibility modifier?
The text was updated successfully, but these errors were encountered: