Uh oh!
There was an error while loading. Please reload this page.
doc: de-clutter documentation with fake-variadics - #101
Merged
Conversation
BennoLossin
previously approved these changes
Jan 18, 2026
BennoLossin
left a comment
Member
There was a problem hiding this comment.
Oh this is great! I didn't know this feature existed. Do you know if there is a similar feature for integers? I'm very much in favor of doing this.
BennoLossin
commented
Feb 24, 2026
Member
I happy to merge this, any last changes @nbdd0121? |
nbdd0121
commented
Feb 24, 2026
MemberAuthor
I want to improve the build process so that docs can still be generated with stable compilers |
BennoLossin
commented
Feb 24, 2026
Member
But that's orthogonal to this, right? |
nbdd0121
commented
Feb 24, 2026
MemberAuthor
Merging this as is would cause the |
BennoLossin
commented
Feb 24, 2026
Member
Oh right, then how about we merge #102 first and then base this on top? |
nbdd0121
commented
Feb 24, 2026
MemberAuthor
Yeah, that's my plan. I'm addressing the feedback of #102 now. |
Uh oh!
There was an error while loading. Please reload this page.
Currently the doc for `Zeroable` and `ZeroableOption` are filled with the generated impl of tuples and fn pointers. Use the internal "fake_variadics" feature to improve the rendered quality. This makes use of an internal feature, however this is of minimal risk as it's for documentation only, not activated during normal build, gated behind `USE_RUSTC_FEATURES`, and can be removed at any time. This feature is already used by serde and bevy to improve documentation quality. For compilers that cannot use this feature, we still hide most generated impls, and the existence of them are hinted by doc comments on the single non-hidden impl. Signed-off-by: Gary Guo <gary@garyguo.net>
BennoLossin
approved these changes
Feb 26, 2026
Uh oh!
There was an error while loading. Please reload this page.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Currently the doc for
ZeroableandZeroableOptionare filled with the generated impl of tuples and fn pointers. Use the internal "fake_variadics" feature to improve the rendered quality.This makes use of an internal feature, however this is of minimal risk as it's for documentation only, not activated during normal build, and can be removed at any time. This feature is already used by serde and bevy to improve documentation quality.
@BennoLossin I'd like to know your opinion on this before sending off to the list. The fake_variadics is optional, we can also just use
doc(hidden)only and leave only one shown. However, it does look nicer with variadics.