search

JelteF / derive_more

Some more derive(Trait) options
MIT License
1.73k stars 123 forks source link

Copy-pasted docs means many derives are still undocumented! #348

Open TheLostLambda opened 6 months ago

TheLostLambda commented 6 months ago

A blocker for 1.0.0, I'd imagine, but shouldn't be too hard to fix!

I found Not / Neg first: https://docs.rs/derive_more/1.0.0-beta.6/derive_more/derive.Neg.html

But it seems to be a broader trend when things are part of the same feature! https://docs.rs/derive_more/1.0.0-beta.6/derive_more/derive.BitAnd.html ... and presumably more!

JelteF commented 6 months ago

Yeah, that's indeed the case. I don't think it's worth the effort to provide specific docs for each, but it would probably be good to prefix those docs with something like:

Neg is derived in the same way as Not, which is described below

tyranron commented 6 months ago

Not a blocker, though.

JelteF commented 6 months ago

@TheLostLambda feel free to provide a PR to update the docs