Skip to content

/rust

New issue

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

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

Already on GitHub? Sign in to your account

rustdocs: missing items in reexported modules #30213

Closed
SimonSapin opened this Issue Dec 4, 2015 · 5 comments

Comments

Assignees

@brson brson

Labels
T-rustdoc regression-from-stable-to-beta
Projects
None yet
Milestone
No milestone
6 participants
@SimonSapin @petrochenkov @alexcrichton @mitaa @brson @sfackler
@SimonSapin
Copy link
Contributor

SimonSapin commented Dec 4, 2015

std::fmt is reexported as:

pub use core_collections::fmt;

This didn’t used to be a problem, http://doc.rust-lang.org/1.4.0/std/fmt/#structs documents a bunch of things. But most of it went missing: http://doc.rust-lang.org/nightly/std/fmt/#structs. (Note that URLs are for 1.4 and current nightly.)

CC @alexcrichton
@petrochenkov

This comment has been minimized.

Sign in to view
Copy link
Contributor

petrochenkov commented Dec 4, 2015

The main suspect is probably #29822, I'll look at what happens.
@alexcrichton

This comment has been minimized.

Sign in to view
Copy link
Member

alexcrichton commented Dec 4, 2015

Thanks for investigating @petrochenkov!

@alexcrichton alexcrichton added the T-rustdoc label Dec 4, 2015

@petrochenkov

This comment has been minimized.

Sign in to view
Copy link
Contributor

petrochenkov commented Dec 5, 2015

Ha, it's not #29822, it's #30043.
cc @arielb1

@petrochenkov petrochenkov referenced this issue Dec 21, 2015

Closed

[beta API Doc] Broken links in `fmt` module #30477

@sfackler sfackler added regression-from-stable-to-beta T-tools labels Dec 21, 2015

@brson brson self-assigned this Jan 7, 2016

@Eljay Eljay referenced this issue Jan 13, 2016

Merged

Fix rustdoc reexports #30870

bors added a commit that referenced this issue Jan 13, 2016

@bors
Auto merge of #30870 - Eljay:issue-30477, r=alexcrichton 
Fixes #30477, #30213.

The loop over reexports used to be a closure before #30043 but it's an iterator now so it should just continue instead of exiting the loop and skipping stuff.

r? @brson
1447ce7
@mitaa

This comment has been minimized.

Sign in to view
Copy link
Contributor

mitaa commented Jan 15, 2016

Fixed by #30870
@alexcrichton

This comment has been minimized.

Sign in to view
Copy link
Member

alexcrichton commented Jan 15, 2016

Thanks @mitaa!

@alexcrichton alexcrichton closed this Jan 15, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.