Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Don't record the root module in the search index #30504

Merged
merged 1 commit into from
Dec 23, 2015
Merged

Don't record the root module in the search index #30504

merged 1 commit into from
Dec 23, 2015

Conversation

mitaa
Copy link
Contributor

@mitaa mitaa commented Dec 21, 2015

fixes #30327

@rust-highfive
Copy link
Collaborator

r? @alexcrichton

(rust_highfive has picked a reviewer for you, use r? to override)

@alexcrichton
Copy link
Member

Perhaps this could just check for index instead of also krate? That'd help avoid the extra const at least

@mitaa
Copy link
Contributor Author

mitaa commented Dec 22, 2015

Yes, this seems to work just as well.

@alexcrichton
Copy link
Member

@bors: r+ 05780ac

bors added a commit that referenced this pull request Dec 23, 2015
@bors
Auto merge of #30504 - mitaa:rdoc_root, r=alexcrichton
3820150 
fixes #30327
@bors
Copy link
Contributor

bors commented Dec 23, 2015

⌛ Testing commit 05780ac with merge 3820150...

@bors
Copy link
Contributor

bors commented Dec 23, 2015

☀️ Test successful - auto-linux-32-nopt-t, auto-linux-32-opt, auto-linux-64-debug-opt, auto-linux-64-nopt-t, auto-linux-64-opt, auto-linux-64-x-android-t, auto-linux-cross-opt, auto-linux-musl-64-opt, auto-mac-32-opt, auto-mac-64-nopt-t, auto-mac-64-opt, auto-win-gnu-32-nopt-t, auto-win-gnu-32-opt, auto-win-gnu-64-nopt-t, auto-win-gnu-64-opt, auto-win-msvc-32-opt, auto-win-msvc-64-opt

@bors bors merged commit 05780ac into rust-lang:master Dec 23, 2015
@mitaa mitaa deleted the rdoc_root branch December 23, 2015 08:57
@mitaa mitaa mentioned this pull request Feb 15, 2016
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@alexcrichton alexcrichton

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Rustdoc: faulty search index entry
4 participants
@mitaa @rust-highfive @alexcrichton @bors