Skip to content

Document From::from impls#137330

Open
TimTheBig wants to merge 22 commits intorust-lang:mainfrom
TimTheBig:document-from
Open

Document From::from impls#137330
TimTheBig wants to merge 22 commits intorust-lang:mainfrom
TimTheBig:document-from

Conversation

@TimTheBig
Copy link

@TimTheBig TimTheBig commented Feb 20, 2025 •
edited by rustbot
Loading

View all comments

This resolves #51430 by documenting all from impls in lib std and core.
I still need someone to look over a few of the comments to see if the style is correct.

Loading
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@programmerjake programmerjake programmerjake left review comments

@blyxyas blyxyas blyxyas left review comments

@oli-obk oli-obk Awaiting requested review from oli-obk

@workingjubilee workingjubilee Awaiting requested review from workingjubilee

+1 more reviewer

@Sky9x Sky9x Sky9x requested changes

Reviewers whose approvals may not affect merge requirements

Assignees

@workingjubilee workingjubilee

Labels

A-docs Area: Documentation for any part of the project, including the compiler, standard library, and tools C-enhancement Category: An issue proposing an enhancement or a PR with one. O-linux Operating system: Linux O-solid Operating System: SOLID O-unix Operating system: Unix-like O-windows Operating system: Windows S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. T-libs Relevant to the library team, which will review and decide on the PR/issue.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Many From implementations are undocumented