-
Notifications
You must be signed in to change notification settings - Fork 205
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
Specify primary constructors #3023
Conversation
A bunch of different declarations can be explored by means of
to see some help and a single example. Try using
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Review response.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Very excited about this potential feature!
581e36f
to
33236bb
Compare
Thanks, @lrhn, very useful feedback! I've closed a bunch of threads that I thought were uncontroversial, and left others open. It is indeed tricky to specify anything in terms of a desugaring step. Thinking about it. ;-) |
Many smaller improvements introduced in response to review comments, PTAL. |
6695c43
to
905ed4f
Compare
a4af6e8
to
66692e7
Compare
The language team agreed to land this proposal at this time. It is in the 'working' directory, which implies that it is a proposal, not a design which has received a final acceptance. |
This PR adds a proposed feature specification for primary constructors, applicable to normal classes as well as inline classes.
This PR has been merged. It added this feature specification.