Replace some desc logic in librustc_lint with article_and_desc#69740
Replace some desc logic in librustc_lint with article_and_desc#69740bors merged 5 commits intorust-lang:masterfrom
Conversation
This comment has been minimized.
This comment has been minimized.
|
The job Click to expand the log.I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
|
☔ The latest upstream changes (presumably #69822) made this pull request unmergeable. Please resolve the merge conflicts. |
This comment has been minimized.
This comment has been minimized.
|
The job Click to expand the log.I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
|
It looks like those functions are called for def_ids that have |
|
The important thing for |
|
@rustbot modify labels: +S-waiting-on-author -S-blocked @eddyb It looks like they are things like EDIT: I have something that seems to work, but I do now see your point about making |
For things that don't have a |
|
@eddyb This is ready now. ui tests passing locally. |
|
@rustbot modify labels: -S-waiting-on-author +S-waiting-on-review |
| @@ -349,7 +349,7 @@ impl MissingDoc { | |||
| id: Option<hir::HirId>, | |||
There was a problem hiding this comment.
Does anything pass None for this? If not you could remove the Option and compute article_and_desc inside here from the HirId.
There was a problem hiding this comment.
Unfortunately, it is passed with a Crate down below, so for now I will keep the option.
|
The job Click to expand the log.I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
|
@bors r+ |
|
📌 Commit d40dff9 has been approved by |
|
The job Click to expand the log.I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
|
⌛ Testing commit d40dff9 with merge c7397479d94c8f7d2e6c6ac68aa10de635d85067... |
|
Oh, it looks like I missed a test and I think bors will fail :( |
|
@bors r+ |
|
📌 Commit 5e8b795 has been approved by |
Replace some desc logic in librustc_lint with article_and_desc r? @eddyb @Centril @matthewjasper Followup to rust-lang#69674 Blocked on rust-lang#69498
Replace some desc logic in librustc_lint with article_and_desc r? @eddyb @Centril @matthewjasper Followup to rust-lang#69674 Blocked on rust-lang#69498
Rollup of 11 pull requests Successful merges: - rust-lang#67761 (Move the dep_graph construction to a dedicated crate.) - rust-lang#69740 (Replace some desc logic in librustc_lint with article_and_desc) - rust-lang#69981 (Evaluate repeat expression lengths as late as possible) - rust-lang#70087 (Remove const eval loop detector) - rust-lang#70242 (Improve E0308 error message wording) - rust-lang#70264 (Fix invalid suggestion on `&mut` iterators yielding `&` references) - rust-lang#70267 (get rid of ConstPropUnsupported; use ZST marker structs instead) - rust-lang#70277 (Remove `ReClosureBound`) - rust-lang#70283 (Add regression test for rust-lang#70155.) - rust-lang#70294 (Account for bad placeholder types in where clauses) - rust-lang#70309 (Clean up E0452 explanation) Failed merges: r? @ghost
r? @eddyb @Centril @matthewjasper
Followup to #69674
Blocked on #69498