Skip to content

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

Merged
rust-bors[bot] merged 1 commit intorust-lang:mainfrom
GuillaumeGomez:add-test-reexport
Mar 4, 2026
Merged

Add regression test for doc(fake_variadic) on reexports#153280
rust-bors[bot] merged 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

⌛ Testing commit ae8890f with merge 8019f1f...

Workflow: https://github.com/rust-lang/rust/actions/runs/22649269407

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
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)
@rust-bors rust-bors bot merged commit d16ec85 into rust-lang:main Mar 4, 2026
11 of 12 checks passed
@rustbot rustbot added this to the 1.96.0 milestone Mar 4, 2026
rust-timer added a commit that referenced this pull request Mar 4, 2026
Rollup merge of #153280 - GuillaumeGomez:add-test-reexport, r=lolbinarycat

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
@GuillaumeGomez GuillaumeGomez deleted the add-test-reexport branch March 4, 2026 09:20
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