Skip to content

Extend 'unimplemented!' API docs to mention new parameter functionality #42628

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Closed
frewsxcv opened this issue Jun 13, 2017 · 5 comments
Closed
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. E-easy Call for participation: Easy difficulty. Experience needed to fix: Not much. Good first issue. E-mentor Call for participation: This issue has a mentor. Use #t-compiler/help on Zulip for discussion. P-medium Medium priority

Comments

@frewsxcv
Copy link
Member

Follow-up to #42155

@frewsxcv frewsxcv added the A-docs Area: Documentation for any part of the project, including the compiler, standard library, and tools label Jun 13, 2017
@frewsxcv frewsxcv changed the title Extened 'unimplemented!' API docs to mention new parameter functionality Extend 'unimplemented!' API docs to mention new parameter functionality Jun 20, 2017
@Mark-Simulacrum Mark-Simulacrum added the C-enhancement Category: An issue proposing an enhancement or a PR with one. label Jul 27, 2017
@steveklabnik steveklabnik added E-easy Call for participation: Easy difficulty. Experience needed to fix: Not much. Good first issue. E-mentor Call for participation: This issue has a mentor. Use #t-compiler/help on Zulip for discussion. P-medium Medium priority labels Aug 30, 2017
@steveklabnik
Copy link
Member

I'd be happy to mentor anyone who wants to take this on! It mostly involves adding a small example to the docs, a great first issue!

@MarkMcCaskey
Copy link
Contributor

@steveklabnik I'd be interested! Thanks!

@toidiu
Copy link
Contributor

toidiu commented Sep 6, 2017

@MarkMcCaskey are you still working on this?

@kennytm
Copy link
Member

kennytm commented Sep 6, 2017

@toidiu "initial changes" implemented as #44206, currently waiting to be merged.

Mark-Simulacrum added a commit to Mark-Simulacrum/rust that referenced this issue Sep 6, 2017
update unimplemented! docs

For rust-lang#42628 (updating docs from changes from rust-lang#42155).

Initial changes made to make `unimplemented!` doc comments look more like `unreachable!` and remove statement about the panic message.

r? @steveklabnik
@toidiu
Copy link
Contributor

toidiu commented Sep 6, 2017

@kennytm oops sry missed that

Mark-Simulacrum added a commit to Mark-Simulacrum/rust that referenced this issue Sep 7, 2017
update unimplemented! docs

For rust-lang#42628 (updating docs from changes from rust-lang#42155).

Initial changes made to make `unimplemented!` doc comments look more like `unreachable!` and remove statement about the panic message.

r? @steveklabnik
# for free to join this conversation on GitHub. Already have an account? # to comment
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. E-easy Call for participation: Easy difficulty. Experience needed to fix: Not much. Good first issue. E-mentor Call for participation: This issue has a mentor. Use #t-compiler/help on Zulip for discussion. P-medium Medium priority
Projects
None yet
Development

No branches or pull requests

6 participants