Conversation
|
r? @davidtwco (rust-highfive has picked a reviewer for you, use r? to override) |
This comment has been minimized.
This comment has been minimized.
This stabilizes the feature described in RFC 2972, which supersedes the earlier RFC 1201. Closes rust-lang#32408 Closes rust-lang#90957
davidtwco
left a comment
There was a problem hiding this comment.
Implementation changes in this pull request looks good to me, it's ready for when the FCP over at #90957 (comment) resolves, assuming no new required changes come out of that process.
|
☔ The latest upstream changes (presumably #94096) made this pull request unmergeable. Please resolve the merge conflicts. |
|
@bstrie The FCP has now completed. Please update this PR. |
|
Ping from triage: @bstrie Could you resolve merge conflicts and make this PR ready? Thanks! |
|
Another gentle ping here? It would be really nice to see this merged. |
|
Based on a quick skim, I think this is blocked on PR #93809, since we do not to stabilize the |
|
👋 Hello, I'm writing this comment in this stabilization PR to notify you, the authors of this PR, that #100591 has been merged, which implemented a change in how features are stabilized. Your PR has been filed before the change, so will likely require modifications in order to comply with the new rules. I recommend you to:
That's it! The If you have any questions, feel free to drop by the zulip stream, or ping me directly in this PR's thread. Thanks! 👋 |
|
With today's stabilization of ( |
|
Mergeing this would definitely boost firmware development in Rust, I'm strongly agree to stabilize this feature. We may expect some reviews but will finally get there. |
|
Going to close this pr as it is blocked on another PR which is still a draft PR and hence this will stay blocked for a long time. |
|
Out of curiosity, what's the other PR? |
|
It was linked a few comments up: #93809 Seems to be stalled since May |
|
Ah, thanks. |
This stabilizes the feature described in RFC 2972, which supercedes the earlier RFC 1201.
Closes #32408
Closes #90957