Skip to content

test: Fixed arg order in rustdoc json test#16968

Merged
weihanglo merged 1 commit intorust-lang:masterfrom
ranger-ross:update-test
May 6, 2026
Merged

test: Fixed arg order in rustdoc json test#16968
weihanglo merged 1 commit intorust-lang:masterfrom
ranger-ross:update-test

Conversation

@ranger-ross
Copy link
Copy Markdown
Member

While rebasing #16807 I notice a new test was added in #16773 that passes -Zbuild-dir-new-layout but the test is asserting the old layout. Which I was initially quite confused by.

After investigating I realized it was due to -Z flags being before and after the command.

cargo -Zbuild-dir-new-layout <command> -Zunstable-options

Moving the -Zunstable-options before the command fixes the issue. (moving -Zbuild-dir-new-layout after the command also works)

Not quite sure if this is a bug or just initiative behavior.
I do not see it documented in https://doc.rust-lang.org/cargo/reference/unstable.html so if its expected I think it might be worth documenting.
Maybe related: #15589

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label May 6, 2026
@rustbot
Copy link
Copy Markdown
Collaborator

rustbot commented May 6, 2026

r? @weihanglo

rustbot has assigned @weihanglo.
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:

  • Owners of files modified in this PR: @ehuss, @epage, @weihanglo
  • @ehuss, @epage, @weihanglo expanded to ehuss, epage, weihanglo
  • Random selection from ehuss, epage, weihanglo

Copy link
Copy Markdown
Member

@weihanglo weihanglo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Probably it was recognized as rustdoc flag. Anyway, thanks!

View changes since this review

@weihanglo weihanglo enabled auto-merge May 6, 2026 11:50
@weihanglo weihanglo added this pull request to the merge queue May 6, 2026
Merged via the queue into rust-lang:master with commit c297c3e May 6, 2026
56 of 58 checks passed
@rustbot rustbot removed the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label May 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants