rust-lang / crater

Run experiments across parts of the Rust ecosystem!
https://crater.rust-lang.org
639 stars 90 forks source link

Improve UI a bit #675

Closed GuillaumeGomez closed 1 year ago

GuillaumeGomez commented 1 year ago

I had a hard time trying to figure out what I was reading. Thanks to @lqd, I was able to understand so I think a little UI improvement wouldn't necessarily be a bad idea. Here is what it looks like with this change:

Screenshot from 2022-12-09 17-43-04

It adds new "header" which provides more context to what is displayed.

GuillaumeGomez commented 1 year ago

Considering there is no rust change in this PR, I assume some clippy lints need some fixes. I'll send another PR for them.

Mark-Simulacrum commented 1 year ago

@bors r+

bors commented 1 year ago

:pushpin: Commit f503c380bd32a6da356fb32ee7100ea2d6bd5830 has been approved by Mark-Simulacrum

It is now in the queue for this repository.

bors commented 1 year ago

:hourglass: Testing commit f503c380bd32a6da356fb32ee7100ea2d6bd5830 with merge e3f0570089eb13fdd3a193ae710cdcf92d23783f...

bors commented 1 year ago

:broken_heart: Test failed - checks-actions

GuillaumeGomez commented 1 year ago

I rebased on https://github.com/rust-lang/crater/pull/676 so CI should be happy now.

GuillaumeGomez commented 1 year ago

@Mark-Simulacrum Fixed CI.

Mark-Simulacrum commented 1 year ago

@bors r+

bors commented 1 year ago

:pushpin: Commit f63b497d6491967956384a924ef60a77b5320def has been approved by Mark-Simulacrum

It is now in the queue for this repository.

bors commented 1 year ago

:hourglass: Testing commit f63b497d6491967956384a924ef60a77b5320def with merge 9365bc77095dcd16785295c69d05c4d2f73543fc...

bors commented 1 year ago

:sunny: Test successful - checks-actions Approved by: Mark-Simulacrum Pushing 9365bc77095dcd16785295c69d05c4d2f73543fc to master...