Skip to content

rustdoc: Remove duplicate "List of all items" #82484

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

Merged
merged 2 commits into from
Feb 26, 2021
Merged

Conversation

bugadani
Copy link
Contributor

Closes #82477

r? @jyn514

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Feb 24, 2021
@jyn514
Copy link
Member

jyn514 commented Feb 24, 2021

Can you add a test case for this so it doesn't regress again?

@jyn514 jyn514 added beta-nominated Nominated for backporting to the compiler in the beta channel. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. T-rustdoc Relevant to the rustdoc team, which will review and decide on the PR/issue. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. labels Feb 24, 2021
@bugadani
Copy link
Contributor Author

Ugh... I don't know about that, this is a rather random piece of string that would be checked. Should we test the whole template?

@jyn514
Copy link
Member

jyn514 commented Feb 24, 2021

Hmm, I guess since this is literal text there's not a ton of point in testing it ... @GuillaumeGomez what do you think?

r=me on the fix though

r? @GuillaumeGomez

@jyn514 jyn514 added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. labels Feb 24, 2021
@GuillaumeGomez
Copy link
Member

You can check that you don't have that string, but yes, please add a test. :)

@jyn514
Copy link
Member

jyn514 commented Feb 24, 2021

@bors r+

@bors
Copy link
Collaborator

bors commented Feb 24, 2021

📌 Commit d3f75eb has been approved by jyn514

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Feb 24, 2021
@camelid
Copy link
Member

camelid commented Feb 25, 2021

This seems completely fine to backport, so 👍 from me.

@GuillaumeGomez
Copy link
Member

@bors: rollup

@apiraino apiraino added the beta-accepted Accepted for backporting to the compiler in the beta channel. label Feb 25, 2021
bors added a commit to rust-lang-ci/rust that referenced this pull request Feb 25, 2021
Rollup of 11 pull requests

Successful merges:

 - rust-lang#82269 (Cleanup `PpMode` and friends)
 - rust-lang#82431 (Set RUST_BACKTRACE=0 when running `treat-err-as-bug` tests)
 - rust-lang#82441 (Fix typo in sanitizer flag in unstable book.)
 - rust-lang#82463 (panic_bounds_checks should be panic_bounds_check)
 - rust-lang#82464 (Update outdated comment in unix Command.)
 - rust-lang#82467 (library: Normalize safety-for-unsafe-block comments)
 - rust-lang#82468 (Move pick_by_value_method docs above function header)
 - rust-lang#82484 (rustdoc: Remove duplicate "List of all items")
 - rust-lang#82502 (Only look for HTML `tidy` when running rustdoc tests)
 - rust-lang#82503 (fix typo in `pre-commit.sh`)
 - rust-lang#82510 (Fix typo in `param_env_reveal_all_normalized`)

Failed merges:

r? `@ghost`
`@rustbot` modify labels: rollup
@bors bors merged commit 8250a25 into rust-lang:master Feb 26, 2021
@rustbot rustbot added this to the 1.52.0 milestone Feb 26, 2021
@cuviper cuviper mentioned this pull request Mar 11, 2021
@cuviper cuviper removed the beta-nominated Nominated for backporting to the compiler in the beta channel. label Mar 11, 2021
@cuviper cuviper modified the milestones: 1.52.0, 1.51.0 Mar 11, 2021
bors added a commit to rust-lang-ci/rust that referenced this pull request Mar 11, 2021
[beta] backports

This backports some beta-accepted PRs and one additional LLVM fix for s390x.

- rustdoc: treat edition 2021 as unstable rust-lang#82207
- Fix popping singleton paths in when generating E0433 rust-lang#82259
- libtest: Fix unwrap panic on duplicate TestDesc rust-lang#82274
- [intra-doc links] Don't check feature gates of items re-exported across crates rust-lang#82295
- rustdoc: Remove duplicate "List of all items" rust-lang#82484
- Substitute erased lifetimes on bad placeholder type rust-lang#82494
- Revert LLVM D81803 because it broke Windows 7 rust-lang#82605
- [SystemZ] Assign the full space for promoted and split outgoing args. rust-lang/llvm-project#95

r? `@Mark-Simulacrum`
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
beta-accepted Accepted for backporting to the compiler in the beta channel. S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. T-rustdoc Relevant to the rustdoc team, which will review and decide on the PR/issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

rustdoc: "List of all items" title shown twice
9 participants