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-json: Document and Test that args can be patterns. #103065

Merged
merged 1 commit into from
Dec 1, 2022

Conversation

aDotInTheVoid
Copy link
Member

No description provided.

@rustbot rustbot added A-rustdoc-json Area: Rustdoc JSON backend T-rustdoc Relevant to the rustdoc team, which will review and decide on the PR/issue. labels Oct 14, 2022
@rustbot
Copy link
Collaborator

rustbot commented Oct 14, 2022

rustdoc-json-types is a public (although nightly-only) API. If possible, consider changing src/librustdoc/json/conversions.rs; otherwise, make sure you bump the FORMAT_VERSION constant.

cc @CraftSpider, @aDotInTheVoid, @Enselic, @obi1kenobi

@rust-highfive
Copy link
Collaborator

r? @CraftSpider

(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 Oct 14, 2022
@aDotInTheVoid
Copy link
Member Author

CraftSpider has retired from the rustdoc team (rust-lang/team#897), rerolling review.

r? rustdoc

@GuillaumeGomez
Copy link
Member

@pietroalbini Do you know why CraftSpider is still being picked by the bot?

@aDotInTheVoid: Thanks!

@bors r+ rollup

@bors
Copy link
Contributor

bors commented Nov 30, 2022

📌 Commit bb04e7e has been approved by GuillaumeGomez

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 Nov 30, 2022
@aDotInTheVoid
Copy link
Member Author

Do you know why CraftSpider is still being picked by the bot?

CraftSpider was picked on Oct 14, when they were still on the team/r+ roation AFAIKT

@GuillaumeGomez
Copy link
Member

Oh I see. Didn't notice this PR was open for so long...

matthiaskrgr added a commit to matthiaskrgr/rust that referenced this pull request Nov 30, 2022
…uillaumeGomez

rustdoc-json: Document and Test that args can be patterns.
bors added a commit to rust-lang-ci/rust that referenced this pull request Dec 1, 2022
…iaskrgr

Rollup of 9 pull requests

Successful merges:

 - rust-lang#103065 (rustdoc-json: Document and Test that args can be patterns.)
 - rust-lang#104865 (Don't overwrite local changes when updating submodules)
 - rust-lang#104895 (Avoid Invalid code suggested when encountering unsatisfied trait bounds in derive macro code)
 - rust-lang#105063 (Rustdoc Json Tests: Don't assume that core::fmt::Debug will always have one item.)
 - rust-lang#105064 (rustdoc: add comment to confusing CSS `main { min-width: 0 }`)
 - rust-lang#105074 (Add Nicholas Bishop to `.mailmap`)
 - rust-lang#105081 (Add a regression test for rust-lang#104322)
 - rust-lang#105086 (rustdoc: clean up sidebar link CSS)
 - rust-lang#105091 (add Tshepang Mbambo to .mailmap)

Failed merges:

r? `@ghost`
`@rustbot` modify labels: rollup
@bors bors merged commit 091f5e1 into rust-lang:master Dec 1, 2022
@rustbot rustbot added this to the 1.67.0 milestone Dec 1, 2022
aDotInTheVoid added a commit to rust-lang/rustdoc-types that referenced this pull request Jan 3, 2023
@aDotInTheVoid aDotInTheVoid deleted the rdj-arg-pattern branch July 4, 2024 19:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-rustdoc-json Area: Rustdoc JSON backend 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.

6 participants