Esteban Kuber
Esteban Kuber
@oli-obk first time I see "test-various" in the job name. Maybe there's a weird interaction in that setup with `// compile-flags: -Zdeduplicate-diagnostics=yes`?
> The Crater run in https://github.com/rust-lang/rust/pull/106060#issuecomment-1364442610 shows that almost all of the breakage is in small unmaintained Github projects. Let's not forget that crater is a source of signal, not...
What is the cost to promote the existing lint to a hard error? It doesn't seem to be adding *too big* of a maintenance burden, does it? I fear people...
Can we get a screenshot to see the highlighting?
First of all, I love the work you're putting on this. I have a couple of concerns. 1) I generally worry about changing the output too drastically all at once,...
You had a good point in the internals thread, about the distinction of `fn item` and `fn pointer` being too subtle and *not highlighted*. Maybe we should also modify that...
HA! I was just looking back on zulip and it was @WaffleLapkin, not you 😅
Just realized that we already show the description for the type, so we could include the fn name in it instead (which might be a bad idea for really long...
An alternative would be to have some JS trickery to implement infinite scrolling on the left, another could be to present a calendar page, or most straight-forward, include a few...
I'm not sure about clippy's lint machinery, but the underlying `DiagnosticBuilder` lets you specify more than one suggestion. If that is available, you could suggest both with an appropriate message...