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

Improvements to intra-doc link macro disambiguators #86523

Merged
merged 3 commits into from
Jun 23, 2021

Conversation

LeSeulArtichaut
Copy link
Contributor

A few small improvements around macro disambiguators:

r? @jyn514 cc @Manishearth @camelid

@rust-highfive
Copy link
Collaborator

Some changes occurred in intra-doc-links.

cc @jyn514

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jun 21, 2021
@rust-log-analyzer

This comment has been minimized.

@LeSeulArtichaut LeSeulArtichaut force-pushed the macros-disambiguators branch 2 times, most recently from f6179cd to 718bc9e Compare June 21, 2021 21:37
@rust-log-analyzer

This comment has been minimized.

@LeSeulArtichaut LeSeulArtichaut force-pushed the macros-disambiguators branch from 718bc9e to 5d6abc9 Compare June 21, 2021 22:00
@jyn514 jyn514 added A-intra-doc-links Area: Intra-doc links, the ability to link to items in docs by name T-rustdoc Relevant to the rustdoc team, which will review and decide on the PR/issue. labels Jun 21, 2021
@jyn514 jyn514 added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Jun 21, 2021
@LeSeulArtichaut LeSeulArtichaut force-pushed the macros-disambiguators branch from 5d6abc9 to 26b44c0 Compare June 22, 2021 09:39
@LeSeulArtichaut LeSeulArtichaut added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. labels Jun 22, 2021
@LeSeulArtichaut
Copy link
Contributor Author

@jyn514 Addressed your comments

@LeSeulArtichaut LeSeulArtichaut force-pushed the macros-disambiguators branch from 26b44c0 to f387e8c Compare June 22, 2021 10:40
@jyn514
Copy link
Member

jyn514 commented Jun 22, 2021

@bors r+

Thanks!

@bors
Copy link
Contributor

bors commented Jun 22, 2021

📌 Commit f387e8c 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 Jun 22, 2021
bors added a commit to rust-lang-ci/rust that referenced this pull request Jun 22, 2021
Rollup of 6 pull requests

Successful merges:

 - rust-lang#86223 (Specify the kind of the item for E0121)
 - rust-lang#86521 (Add comments around code where ordering is important due for panic-safety)
 - rust-lang#86523 (Improvements to intra-doc link macro disambiguators)
 - rust-lang#86542 (Line numbers aligned with content)
 - rust-lang#86549 (Add destructuring example of E0508)
 - rust-lang#86557 (Update books)

Failed merges:

 - rust-lang#86548 (Fix crate filter search reset)

r? `@ghost`
`@rustbot` modify labels: rollup
@bors bors merged commit f9ebf1e into rust-lang:master Jun 23, 2021
@rustbot rustbot added this to the 1.55.0 milestone Jun 23, 2021
@LeSeulArtichaut LeSeulArtichaut deleted the macros-disambiguators branch June 23, 2021 08:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-intra-doc-links Area: Intra-doc links, the ability to link to items in docs by name 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.

Intra-doc links don't accept all macro delimiters Intra-doc link foo!() is rendered without the parentheses
6 participants