-
Notifications
You must be signed in to change notification settings - Fork 12.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
rustbuild: Run debuginfo tests by default #38471
Conversation
r? @brson (rust_highfive has picked a reviewer for you, use r? to override) |
This unfortunately may not land due to this bug, but I figure we can throw at bors and see what happens. |
@bors r+ p=1 |
📌 Commit d631367 has been approved by |
⌛ Testing commit d631367 with merge 59858d7... |
Yikes! |
💔 Test failed - auto-linux-64-x-android-t |
@bors: retry |
⌛ Testing commit d631367 with merge 848fe2c... |
💔 Test failed - auto-linux-64-x-android-t |
This fixes an accidental regression in rustbuild which stopped running debuginfo tests by default. Here we flag the test suites as `default(true)` to ensure that they're run on bots, for example.
d631367
to
70ef94d
Compare
@bors: r=brson |
📌 Commit 70ef94d has been approved by |
…=brson rustbuild: Run debuginfo tests by default This fixes an accidental regression in rustbuild which stopped running debuginfo tests by default. Here we flag the test suites as `default(true)` to ensure that they're run on bots, for example.
⌛ Testing commit 70ef94d with merge a4e3e1a... |
@bors: retry force clean
|
Rollup of 29 pull requests - Successful merges: #37761, #38006, #38131, #38150, #38158, #38171, #38208, #38215, #38236, #38245, #38289, #38302, #38315, #38346, #38388, #38395, #38398, #38418, #38432, #38451, #38463, #38468, #38470, #38471, #38472, #38478, #38486, #38493, #38498 - Failed merges: #38271, #38483
This fixes an accidental regression in rustbuild which stopped running debuginfo
tests by default. Here we flag the test suites as
default(true)
to ensure thatthey're run on bots, for example.