Skip to content

rustdoc: Simplify some document functions #84347

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

Merged
merged 1 commit into from
Apr 20, 2021

Conversation

Swatinem
Copy link
Contributor

  • Remove prefix param of document_short/full, render_markdown, as its always an empty string.
  • Remove Option wrapping of document_short parent, as its always Some.

* Remove `prefix` param of `document_short/full`, `render_markdown`, as its always an empty string.
* Remove `Option` wrapping of `document_short` `parent`, as its always `Some`.
@rust-highfive
Copy link
Contributor

r? @jyn514

(rust-highfive has picked a reviewer for you, use r? to override)

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Apr 19, 2021
@jyn514 jyn514 added C-cleanup Category: PRs that clean code up or issues documenting cleanup. T-rustdoc Relevant to the rustdoc team, which will review and decide on the PR/issue. labels Apr 19, 2021
@jyn514
Copy link
Member

jyn514 commented Apr 19, 2021

@bors r+

@bors
Copy link
Collaborator

bors commented Apr 19, 2021

📌 Commit d93d77b 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 Apr 19, 2021
@bors
Copy link
Collaborator

bors commented Apr 19, 2021

⌛ Testing commit d93d77b with merge 500b64b84025620bc02f0a6fb0b3800f168100dd...

@bors
Copy link
Collaborator

bors commented Apr 20, 2021

💔 Test failed - checks-actions

@bors bors added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. labels Apr 20, 2021
@rust-log-analyzer
Copy link
Collaborator

A job failed! Check out the build log: (web) (plain)

Click to see the possible cause of the failure (guessed by this bot)

@jyn514
Copy link
Member

jyn514 commented Apr 20, 2021

aarch64-gnu failed without logs :/

@bors retry

@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 Apr 20, 2021
bors added a commit to rust-lang-ci/rust that referenced this pull request Apr 20, 2021
Rollup of 4 pull requests

Successful merges:

 - rust-lang#84337 (Clarify the difference between insert and get_or_insert)
 - rust-lang#84340 (rustdoc: Show nag box on IE11)
 - rust-lang#84345 (Remove comment about doc hack.)
 - rust-lang#84347 (rustdoc: Simplify some document functions)

Failed merges:

r? `@ghost`
`@rustbot` modify labels: rollup
@bors bors merged commit 868de80 into rust-lang:master Apr 20, 2021
@bors
Copy link
Collaborator

bors commented Apr 20, 2021

⌛ Testing commit d93d77b with merge 7d0132a...

@rustbot rustbot added this to the 1.53.0 milestone Apr 20, 2021
@Swatinem Swatinem deleted the simplify-document-fns branch November 3, 2021 12:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-cleanup Category: PRs that clean code up or issues documenting cleanup. 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.

6 participants