Skip to content

rustdoc: Rename def_id into item_id when the type is ItemId for readability - #96118

Merged
bors merged 1 commit into
rust-lang:masterfrom
GuillaumeGomez:cleanup-def-id-item-id
Apr 16, 2022
Merged

rustdoc: Rename def_id into item_id when the type is ItemId for readability#96118
bors merged 1 commit into
rust-lang:masterfrom
GuillaumeGomez:cleanup-def-id-item-id

Conversation

@GuillaumeGomez

Copy link
Copy Markdown
Member

As @notriddle mentioned in #96091, the field name is inaccurate. This PR fixes it by renaming it accordingly to its real type.

r? @notriddle

@rust-highfive

Copy link
Copy Markdown
Contributor

Some changes occurred in clean/types.rs.

cc @camelid

@rustbotrustbot added the T-rustdoc Relevant to the rustdoc team, which will review and decide on the PR/issue. label Apr 16, 2022
@rust-highfiverust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Apr 16, 2022
@GuillaumeGomezGuillaumeGomez changed the title Rename def_id into item_id when the type is ItemId for readabilityrustdoc: Rename def_id into item_id when the type is ItemId for readabilityApr 16, 2022
@notriddle

Copy link
Copy Markdown
Contributor

@bors r+ rollup=always

@bors

bors commented Apr 16, 2022

Copy link
Copy Markdown
Collaborator

📌 Commit b1e6211 has been approved by notriddle

@borsbors 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 16, 2022
@Dylan-DPCDylan-DPC mentioned this pull request Apr 16, 2022
bors added a commit to rust-lang-ci/rust that referenced this pull request Apr 16, 2022
Rollup of 7 pull requests
Successful merges:
- rust-lang#94985 (Parse inner attributes on inline const block)
- rust-lang#95006 (Reject `#[thread_local]` attribute on non-static items)
- rust-lang#95426 (Include Refs in Valtree Creation)
- rust-lang#95908 (Inline `shallow_resolve_ty` into `ShallowResolver`)
- rust-lang#96058 (separate flock implementations into separate modules)
- rust-lang#96088 (Update mdbook)
- rust-lang#96118 (rustdoc: Rename `def_id` into `item_id` when the type is `ItemId` for readability)
Failed merges:
r? `@ghost`
`@rustbot` modify labels: rollup
@bors
bors merged commit 10e0db5 into rust-lang:masterApr 16, 2022
@rustbotrustbot added this to the 1.62.0 milestone Apr 16, 2022
@GuillaumeGomez
GuillaumeGomez deleted the cleanup-def-id-item-id branch April 16, 2022 21:40
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-borsStatus: Waiting on bors to run and complete tests. Bors will change the label on completion.T-rustdocRelevant 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.

5 participants

@GuillaumeGomez@rust-highfive@notriddle@bors@rustbot