Remove driver_lint_caps#152852
Conversation
|
The rustc-dev-guide subtree was changed. If this PR only touches the dev guide consider submitting a PR directly to rust-lang/rustc-dev-guide otherwise thank you for updating the dev guide with your changes. |
This comment has been minimized.
This comment has been minimized.
d66dd3d to
2aa9c6c
Compare
This comment has been minimized.
This comment has been minimized.
2aa9c6c to
baef897
Compare
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
baef897 to
031197e
Compare
This comment has been minimized.
This comment has been minimized.
031197e to
3934a20
Compare
|
This PR was rebased onto a different main commit. Here's a range-diff highlighting what actually changed. Rebasing is a normal part of keeping PRs up to date, so no action is needed—this note is just to help reviewers. |
This comment has been minimized.
This comment has been minimized.
It was only used by rustdoc and doesn't seem like it was necessary there. No tests fail at least.
3934a20 to
379caad
Compare
|
I wrote a couple of tests for this functionality while reviewing it. I didn't find any problems, but I'd like to put together more test cases for this functionality, too.
@bors r+ |
This comment has been minimized.
This comment has been minimized.
Remove driver_lint_caps It was only used by rustdoc and doesn't seem like it was necessary there. No tests fail at least. r? rust-lang/rustdoc
|
💔 Test for 7bb6818 failed: CI. Failed job:
|
|
A job failed! Check out the build log: (web) (plain enhanced) (plain) Click to see the possible cause of the failure (guessed by this bot) |
|
Nothing in this change seems like it would cause installing /tmp/build-x86_64-fortanix-unknown-sgx-toolchain.sh to fail. @bors retry |
This comment has been minimized.
This comment has been minimized.
Remove driver_lint_caps It was only used by rustdoc and doesn't seem like it was necessary there. No tests fail at least. r? rust-lang/rustdoc
|
💔 Test for 749f898 failed: CI. Failed job:
|
|
A job failed! Check out the build log: (web) (plain enhanced) (plain) Click to see the possible cause of the failure (guessed by this bot) |
…notriddle Remove driver_lint_caps It was only used by rustdoc and doesn't seem like it was necessary there. No tests fail at least. r? rust-lang/rustdoc
Rollup of 6 pull requests Successful merges: - #152852 (Remove driver_lint_caps) - #156121 (compiler: suggest `.collect()` when `String` is expected and `Iterator` is found) - #156518 ([style] rustfmt `match`es with comments in or-patterns) - #156596 (Split `LintExpectationId`s) - #156577 (Test EII UI tests with prefer-dynamic) - #156633 (Add regression test for issue #41261)
This comment has been minimized.
This comment has been minimized.
Remove driver_lint_caps It was only used by rustdoc and doesn't seem like it was necessary there. No tests fail at least. r? rust-lang/rustdoc
|
💔 Test for fd90000 failed: CI. Failed jobs:
|
|
A job failed! Check out the build log: (web) (plain enhanced) (plain) Click to see the possible cause of the failure (guessed by this bot) |
View all comments
It was only used by rustdoc and doesn't seem like it was necessary there. No tests fail at least.
r? rust-lang/rustdoc