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.

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

rustdoc: Rename "object safe" to "dyn compatible" #131594

Merged
merged 1 commit into from
Oct 16, 2024

Conversation

fmease
Copy link
Member

@fmease fmease commented Oct 12, 2024

Supersedes #126554:

  1. In line with T-lang's latest resolution.
  2. More comprehensive: Not only updates user-facing text but also source code.

Part of #130852.

Doesn't update rustdoc-JSON (will be filed separately. #131595).

r? @notriddle (rust-lang/lang-team#286) @GuillaumeGomez (for visibility)

@rustbot
Copy link
Collaborator

rustbot commented Oct 12, 2024

Some changes occurred in src/librustdoc/clean/types.rs

cc @camelid

@rustbot rustbot added A-rustdoc-json Area: Rustdoc JSON backend S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-rustdoc Relevant to the rustdoc team, which will review and decide on the PR/issue. labels Oct 12, 2024
@GuillaumeGomez
Copy link
Member

Apart from notriddle's comments, looks good to me. :)

@fmease fmease force-pushed the rustdoc-mv-obj-safe-dyn-compat branch from 41751a4 to d707530 Compare October 16, 2024 12:47
@rustbot rustbot added the A-rustdoc-json Area: Rustdoc JSON backend label Oct 16, 2024
@fmease fmease removed the A-rustdoc-json Area: Rustdoc JSON backend label Oct 16, 2024
@fmease fmease force-pushed the rustdoc-mv-obj-safe-dyn-compat branch from d707530 to 8a6eb2e Compare October 16, 2024 12:49
@rustbot rustbot added the A-rustdoc-json Area: Rustdoc JSON backend label Oct 16, 2024
@fmease fmease removed the A-rustdoc-json Area: Rustdoc JSON backend label Oct 16, 2024
@fmease
Copy link
Member Author

fmease commented Oct 16, 2024

Suggestions applied. Ready.

@fmease fmease force-pushed the rustdoc-mv-obj-safe-dyn-compat branch from 8a6eb2e to 6d82559 Compare October 16, 2024 13:54
@rustbot rustbot added the A-rustdoc-json Area: Rustdoc JSON backend label Oct 16, 2024
@fmease fmease removed the A-rustdoc-json Area: Rustdoc JSON backend label Oct 16, 2024
@notriddle
Copy link
Contributor

@bors r+

@bors
Copy link
Contributor

bors commented Oct 16, 2024

📌 Commit 6d82559 has been approved by notriddle

It is now in the queue for this repository.

@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 Oct 16, 2024
bors added a commit to rust-lang-ci/rust that referenced this pull request Oct 16, 2024
…iaskrgr

Rollup of 8 pull requests

Successful merges:

 - rust-lang#130822 (Add `from_ref` and `from_mut` constructors to `core::ptr::NonNull`.)
 - rust-lang#131381 (Implement edition 2024 match ergonomics restrictions)
 - rust-lang#131594 (rustdoc: Rename "object safe" to "dyn compatible")
 - rust-lang#131686 (Add fast-path when computing the default visibility)
 - rust-lang#131699 (Try to improve error messages involving aliases in the solver)
 - rust-lang#131757 (Ignore lint-non-snake-case-crate#proc_macro_ on targets without unwind)
 - rust-lang#131783 (Fix explicit_iter_loop in rustc_serialize)
 - rust-lang#131788 (Fix mismatched quotation mark)

r? `@ghost`
`@rustbot` modify labels: rollup
@bors bors merged commit 7047748 into rust-lang:master Oct 16, 2024
6 checks passed
@rustbot rustbot added this to the 1.84.0 milestone Oct 16, 2024
@fmease fmease deleted the rustdoc-mv-obj-safe-dyn-compat branch October 16, 2024 20:51
rust-timer added a commit to rust-lang-ci/rust that referenced this pull request Oct 16, 2024
Rollup merge of rust-lang#131594 - fmease:rustdoc-mv-obj-safe-dyn-compat, r=notriddle

rustdoc: Rename "object safe" to "dyn compatible"

Supersedes rust-lang#126554:

1. In line with [T-lang's latest resolution](rust-lang/lang-team#286 (comment)).
2. More comprehensive: Not only updates user-facing text but also source code.

Part of rust-lang#130852.

Doesn't update rustdoc-JSON (will be filed separately).

r? `@notriddle` (rust-lang/lang-team#286) `@GuillaumeGomez` (for visibility)
bors added a commit to rust-lang-ci/rust that referenced this pull request Oct 17, 2024
…compat, r=aDotInTheVoid

rustdoc-JSON: Rename "object safe" to "dyn compatible"

~~Blocked: Sits atop rust-lang#131594. Only the last commit is relevant.~~ (rebased)

Part of rust-lang#130852.

r? aDotInTheVoid or rustdoc
bors added a commit to rust-lang-ci/rust that referenced this pull request Oct 17, 2024
…compat, r=aDotInTheVoid

rustdoc-JSON: Rename "object safe" to "dyn compatible"

~~Blocked: Sits atop rust-lang#131594. Only the last commit is relevant.~~ (rebased)

Part of rust-lang#130852.

r? aDotInTheVoid or rustdoc
matthiaskrgr added a commit to matthiaskrgr/rust that referenced this pull request Oct 17, 2024
…n-compat, r=aDotInTheVoid

rustdoc-JSON: Rename "object safe" to "dyn compatible"

~~Blocked: Sits atop rust-lang#131594. Only the last commit is relevant.~~ (rebased)

Part of rust-lang#130852.

r? aDotInTheVoid or rustdoc
rust-timer added a commit to rust-lang-ci/rust that referenced this pull request Oct 17, 2024
Rollup merge of rust-lang#131595 - fmease:rustdoc-json-mv-obj-safe-dyn-compat, r=aDotInTheVoid

rustdoc-JSON: Rename "object safe" to "dyn compatible"

~~Blocked: Sits atop rust-lang#131594. Only the last commit is relevant.~~ (rebased)

Part of rust-lang#130852.

r? aDotInTheVoid or rustdoc
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
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.

5 participants