Move mini-macro to tests/ui/auxilary #7285
Conversation
|
r? @Manishearth (rust-highfive has picked a reviewer for you, use r? to override) |
rust-highfive
added
the
S-waiting-on-review
| } | ||
|
|
||
| #[derive(ClippyMiniMacroTest, Debug)] | ||
| struct Foo; |
There was a problem hiding this comment.
Deleted this for convenience since /crashes has a separate auxilary folder. The test is redundant and history shows no evidence that it ever produced an ICE (see #3743).
There was a problem hiding this comment.
I also can't find the reason why this exists. Probably to check that the derive doesn't produce an ICE. But since we use this in other tests too, deleting this test should be fine.
| } | ||
|
|
||
| #[derive(ClippyMiniMacroTest, Debug)] | ||
| struct Foo; |
There was a problem hiding this comment.
I also can't find the reason why this exists. Probably to check that the derive doesn't produce an ICE. But since we use this in other tests too, deleting this test should be fine.
|
@bors r+ Thanks! This is long overdue. |
|
📌 Commit 6c54f61 has been approved by |
|
☀️ Test successful - checks-action_dev_test, checks-action_remark_test, checks-action_test |
changelog: none
Merges
/mini-macrointo/tests/ui/auxilary/proc_macro_derive.rs.The mini-macro crate is an artifact of the distant past. A lot has changed (#2284) and it doesn't make sense as a top-level crate anymore. Especially since we can use the auxilary folder to accompolish the same thing.