Skip to content

Add missing num_internals feature gate to coretests/benches#154132

Merged
rust-bors[bot] merged 1 commit intorust-lang:mainfrom
bjorn3:fix_missing_feature_gate
Mar 24, 2026
Merged

Add missing num_internals feature gate to coretests/benches#154132
rust-bors[bot] merged 1 commit intorust-lang:mainfrom
bjorn3:fix_missing_feature_gate

Conversation

@bjorn3
Copy link
Copy Markdown
Member

@bjorn3 bjorn3 commented Mar 20, 2026

@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-libs Relevant to the library team, which will review and decide on the PR/issue. labels Mar 20, 2026
@rustbot
Copy link
Copy Markdown
Collaborator

rustbot commented Mar 20, 2026

r? @jieyouxu

rustbot has assigned @jieyouxu.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

Why was this reviewer chosen?

The reviewer was selected based on:

  • Fallback group: @Mark-Simulacrum, @jieyouxu
  • @Mark-Simulacrum, @jieyouxu expanded to Mark-Simulacrum, jieyouxu
  • Random selection from Mark-Simulacrum, jieyouxu

@lqd
Copy link
Copy Markdown
Member

lqd commented Mar 20, 2026

2 questions:

  • the patch has two changes, but this PR has one?
  • if this feature gate is only used in the cg_clif benches/tests, we should say as such in a comment, otherwise someone could remove it as being unused, right?

@bjorn3
Copy link
Copy Markdown
Member Author

bjorn3 commented Mar 20, 2026

the patch has two changes, but this PR has one?

The other change was already done upstream by someone else.

if this feature gate is only used in the cg_clif benches/tests, we should say as such in a comment, otherwise someone could remove it as being unused, right?

It is not. I suspect the reason it doesn't fail in Rust's CI is benches not being tested at all in CI.

@rust-log-analyzer

This comment has been minimized.

@bjorn3 bjorn3 force-pushed the fix_missing_feature_gate branch from 482c64c to 5ef93c1 Compare March 20, 2026 11:22
@jieyouxu
Copy link
Copy Markdown
Member

r? libs

@rustbot rustbot assigned Mark-Simulacrum and unassigned jieyouxu Mar 20, 2026
@Mark-Simulacrum
Copy link
Copy Markdown
Member

@bors r+

@rust-bors
Copy link
Copy Markdown
Contributor

rust-bors bot commented Mar 24, 2026

📌 Commit 5ef93c1 has been approved by Mark-Simulacrum

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 24, 2026
rust-bors bot pushed a commit that referenced this pull request Mar 24, 2026
Rollup of 10 pull requests

Successful merges:

 - #153964 (Fix `doc_cfg` not working as expected on trait impls)
 - #153979 (Rename various query cycle things.)
 - #154132 (Add missing num_internals feature gate to coretests/benches)
 - #154153 (core: Implement `unchecked_funnel_{shl,shr}`)
 - #154236 (Clean up query-forcing functions)
 - #154252 (Don't store current-session side effects in `OnDiskCache`)
 - #154017 ( Fix invalid add of duplicated call locations for the rustdoc scraped examples feature)
 - #154163 (enzyme submodule update)
 - #154264 (Update books)
 - #154282 (rustc-dev-guide subtree update)
@rust-bors rust-bors bot merged commit 355b4f0 into rust-lang:main Mar 24, 2026
11 checks passed
@rustbot rustbot added this to the 1.96.0 milestone Mar 24, 2026
@bjorn3 bjorn3 deleted the fix_missing_feature_gate branch March 24, 2026 09:00
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-libs Relevant to the library 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