Skip to content

fix docs for inherent str constructors#137126

Merged
bors merged 1 commit intorust-lang:masterfrom
m4rch3n1ng:fix-inherent-str-docs
Feb 19, 2025
Merged

fix docs for inherent str constructors#137126
bors merged 1 commit intorust-lang:masterfrom
m4rch3n1ng:fix-inherent-str-docs

Conversation

@m4rch3n1ng
Copy link
Contributor

related to #131114

when implementing inherent str constructors in #136517, i forgot to change the docs, so the code examples still imported the std::str module and used the constructor from there, instead of using "itself" (the inherent constructor).

@rustbot
Copy link
Collaborator

rustbot commented Feb 16, 2025

r? @Amanieu

rustbot has assigned @Amanieu.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-libs Relevant to the library team, which will review and decide on the PR/issue. labels Feb 16, 2025
@Amanieu
Copy link
Member

Amanieu commented Feb 18, 2025

@bors r+ rollup

@bors
Copy link
Collaborator

bors commented Feb 18, 2025

📌 Commit 345c313 has been approved by Amanieu

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 Feb 18, 2025
@bors bors merged commit ac3b179 into rust-lang:master Feb 19, 2025
6 checks passed
@rustbot rustbot added this to the 1.87.0 milestone Feb 19, 2025
@m4rch3n1ng m4rch3n1ng deleted the fix-inherent-str-docs branch February 19, 2025 01:14
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-libs Relevant to the library team, which will review and decide on the PR/issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants