-
Notifications
You must be signed in to change notification settings - Fork 529
Add docs for field default values #1766
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: master
Are you sure you want to change the base?
Conversation
cc @estebank |
@bluebear94: You've been contributing a number of things recently. Thanks for that. We're on the spec call at the moment, and would be interested in talking more with you. If you're able to join us on Zulip, that'd be great. You can then reach out to |
@estebank In particular, I'm curious what the status of this feature is in terms of how close it is to stabilization? Is this something you can help with review? I'm generally shy from doing reviews if there are still changes forthcoming, and I have no context on this feature. |
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.
From a compiler perspective, it's probably still worth baking the feature for some time, since it's still pretty new implementation-wise. But I don't think it's going to change much, so it's probably fine to keep this open and tagged as waiting for stabilization and revisit it for tweaks when the stabilization happens.
This comment has been minimized.
This comment has been minimized.
252b208
to
9e4a554
Compare
This comment has been minimized.
This comment has been minimized.
9e4a554
to
760338b
Compare
Tracking issue: rust-lang/rust#132162