Skip to content

Conversation

kornelski
Copy link
Contributor

C-flavored documentation aliases for quicksort and binary search

@rust-highfive
Copy link
Contributor

r? @Mark-Simulacrum

(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 23, 2021
@jyn514
Copy link
Member

jyn514 commented Apr 23, 2021

r? @m-ou-se
cc #81989, #79943

@GuillaumeGomez GuillaumeGomez added the A-doc-alias Area: `#[doc(alias)]` label Apr 23, 2021
@joshtriplett joshtriplett added the T-libs-api Relevant to the library API team, which will review and decide on the PR/issue. label May 5, 2021
@m-ou-se
Copy link
Member

m-ou-se commented May 5, 2021

Hey, thanks for your PR.

Right now it's not clear what direction we want to go with doc aliases. Since adding every possible equivalent function/type from every popular language quickly gets out of hand, we decided to put a stop to adding new such aliases until we have a policy/guideline in place for them.

See also
#81989 (comment)
#81988 (comment)
https://rust-lang.zulipchat.com/#narrow/stream/219381-t-libs/topic/doc.20aliases

Closing this PR for now. Feel free to re-open once we have a policy and this still fits within it.

@m-ou-se m-ou-se closed this May 5, 2021
@kornelski kornelski deleted the docalias branch November 11, 2023 22:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-doc-alias Area: `#[doc(alias)]` S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-libs-api Relevant to the library API team, which will review and decide on the PR/issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants