Yacin Tmimi
Yacin Tmimi
> No, I hadn't considered that. Got it. > I'm personally unsure of the usecase for `wrap_comments` itself, I've only ever thought the doc-comment variant was useful, so I'm not...
That's great! Thanks for making the change. I took a really quick look and I'm wondering if it would be possible to implement this without any proc macro changes? I...
No worries. What version of rustfmt are you using?
@aDifferentJT Do you also have an input snippet that we could run rustfmt on to reproduce the issue?
@aDifferentJT I don't think this is an issue with a particular code snippet. If I had to guess Id say changes made in https://github.com/rust-lang/rust/pull/121085 are causing this issue.
were you running rustfmt on a public repo? That might be enough to reproduce the issue.
Perfect! I'm also able to reproduce the bug using the snippet. Thank you very much! I think this will help us figure out what's going on.
https://github.com/rust-lang/rust/pull/121487 doesn't explicitly reference this issue, but I'm hoping that it also resolves the problem 🤞🏼
@aDifferentJT when you have a moment can you confirm that this is resolved on the latest nightly (rustfmt 1.7.0-nightly (7d3702e4 2024-03-06)? I just ran your input snippet with a few...
Probably better not to `panic!`. Also, I think the tests are failing because you need to run rustfmt on itself 😅. The `self_test` makes sure that rustfmt is formatted properly.