Skip to content

test: add test case for verify-project with invalid TOML#16640

Merged
weihanglo merged 1 commit intorust-lang:masterfrom
raushan728:test/verify-project-invalid-toml
Feb 14, 2026
Merged

test: add test case for verify-project with invalid TOML#16640
weihanglo merged 1 commit intorust-lang:masterfrom
raushan728:test/verify-project-invalid-toml

Conversation

@raushan728
Copy link
Contributor

Adds a test case for 'cargo verify-project' with invalid TOML syntax.

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

rustbot commented Feb 13, 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
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.

Thanks for the contribution! I understand this wasn't covered by any of the existing tests for cargo verify-project, but I'm curious where did you come across this issue?

View changes since this review

@raushan728
Copy link
Contributor Author

Was reviewing the tests and saw this case was missing so added it

@weihanglo weihanglo added this pull request to the merge queue Feb 14, 2026
Merged via the queue into rust-lang:master with commit f801f10 Feb 14, 2026
29 checks passed
@rustbot rustbot removed the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Feb 14, 2026
@raushan728 raushan728 deleted the test/verify-project-invalid-toml branch February 14, 2026 07:38
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