Documentation change to macros.rs for includes!#36404
Conversation
|
Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @aturon (or someone else) soon. If any changes to this PR are deemed necessary, please add them as extra commits. This ensures that the reviewer can see what has changed since they last reviewed the code. Due to the way GitHub handles out-of-date commits, this should also make it reasonably obvious what issues have or haven't been addressed. Large or tricky changes may require several passes of review and changes. Please see the contribution instructions for more information. |
includes!
src/libstd/macros.rs
Outdated
There was a problem hiding this comment.
I believe the file is can only be parsed as an expression or an item -- c.f. this impl.
d57c163 to
3660a79
Compare
|
Apologies for the very delayed review, and thanks for the PR! @bors: r+ rollup |
|
@bors: r+ rollup |
|
📌 Commit 3660a79 has been approved by |
Documentation change to macros.rs for `includes!` I'm not sure if this documentation is clear or extensive enough, but this is just to get started on the problem, fixes issue #36387.
libcore documentation for builtin macros Fixes: #36272 Additionally I've edited docstring for `include!` a bit. (related PR #36404) Unfortunately it seems there is no sane way to reexport empty macros definitions for their docstrings. To avoid copying the whole documentation for builtin macros I've only copied description and added links to `std` macro pages.
I'm not sure if this documentation is clear or extensive enough, but this is just to get started on the problem, fixes issue #36387.