Skip to content

Add regression test for doc(fake_variadic) on reexports#153280

Open
GuillaumeGomez wants to merge 1 commit intorust-lang:mainfrom
GuillaumeGomez:add-test-reexport
Open

Add regression test for doc(fake_variadic) on reexports#153280
GuillaumeGomez wants to merge 1 commit intorust-lang:mainfrom
GuillaumeGomez:add-test-reexport

Conversation

@GuillaumeGomez
Copy link
Member

Follow-up of #153136. Out of the four doc attributes remaining to be tested (fake_variadic, keyword, attribute and notable_trait), only fake_variadic could have been impacted by reexports:

  • attribute and keyword are not supposed to be reexported (they're supposed to be used on private modules)
  • notable_trait is applied to a trait, so whether it's reexported or not, it still works.

r? @lolbinarycat

@rustbot rustbot added 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. T-rustdoc-frontend Relevant to the rustdoc-frontend team, which will review and decide on the web UI/UX output. labels Mar 2, 2026
@lolbinarycat
Copy link
Contributor

Thanks!

@bors r+

@rust-bors
Copy link
Contributor

rust-bors bot commented Mar 3, 2026

📌 Commit ae8890f has been approved by lolbinarycat

It is now in the queue for this repository.

@rust-bors rust-bors bot 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 Mar 3, 2026
JonathanBrouwer added a commit to JonathanBrouwer/rust that referenced this pull request Mar 3, 2026
…r=lolbinarycat

Add regression test for `doc(fake_variadic)` on reexports

Follow-up of rust-lang#153136. Out of the four doc attributes remaining to be tested (`fake_variadic`, `keyword`, `attribute` and `notable_trait`), only `fake_variadic` could have been impacted by reexports:

* `attribute` and `keyword` are not supposed to be reexported (they're supposed to be used on private modules)
* `notable_trait` is applied to a trait, so whether it's reexported or not, it still works.

r? @lolbinarycat
rust-bors bot pushed a commit that referenced this pull request Mar 3, 2026
…uwer

Rollup of 5 pull requests

Successful merges:

 - #153280 (Add regression test for `doc(fake_variadic)` on reexports)
 - #153358 (Boundary tests for various Duration-float operations)
 - #153272 (Add `Path::absolute` method as alias for `std::path::absolute`)
 - #153295 (update panicking() docs for panic=abort)
 - #153352 (Migration of `LintDiagnostic` - part 6)
@rust-bors
Copy link
Contributor

rust-bors bot commented Mar 4, 2026

rust-bors bot pushed a commit that referenced this pull request Mar 4, 2026
Add regression test for `doc(fake_variadic)` on reexports

Follow-up of #153136. Out of the four doc attributes remaining to be tested (`fake_variadic`, `keyword`, `attribute` and `notable_trait`), only `fake_variadic` could have been impacted by reexports:

* `attribute` and `keyword` are not supposed to be reexported (they're supposed to be used on private modules)
* `notable_trait` is applied to a trait, so whether it's reexported or not, it still works.

r? @lolbinarycat
Zalathar added a commit to Zalathar/rust that referenced this pull request Mar 4, 2026
…r=lolbinarycat

Add regression test for `doc(fake_variadic)` on reexports

Follow-up of rust-lang#153136. Out of the four doc attributes remaining to be tested (`fake_variadic`, `keyword`, `attribute` and `notable_trait`), only `fake_variadic` could have been impacted by reexports:

* `attribute` and `keyword` are not supposed to be reexported (they're supposed to be used on private modules)
* `notable_trait` is applied to a trait, so whether it's reexported or not, it still works.

r? @lolbinarycat
@Zalathar
Copy link
Member

Zalathar commented Mar 4, 2026

@bors yield (to enclosing rollup)

@rust-bors
Copy link
Contributor

rust-bors bot commented Mar 4, 2026

Auto build cancelled. Cancelled workflows:

The next pull request likely to be tested is #153377.

rust-bors bot pushed a commit that referenced this pull request Mar 4, 2026
Rollup of 8 pull requests

Successful merges:

 - #153280 (Add regression test for `doc(fake_variadic)` on reexports)
 - #153302 (x86: reserve `bl` and `bh` registers to match `rbx`)
 - #153358 (Boundary tests for various Duration-float operations)
 - #153048 (Improve irrefutable let-else lint wording)
 - #153258 (diag: Suppress `.clone()` suggestion inside derive macro expansions)
 - #153272 (Add `Path::absolute` method as alias for `std::path::absolute`)
 - #153295 (update panicking() docs for panic=abort)
 - #153352 (Migration of `LintDiagnostic` - part 6)
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. T-rustdoc-frontend Relevant to the rustdoc-frontend team, which will review and decide on the web UI/UX output.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants