New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
internal: Bump rustc-perf
checkout for metrics, replace webrender, diesel and ripgrep with newer versions
#15552
Conversation
r? @lnicola (since you've seen it already :^) Gonna check how much time we get back to spent on new targets after this has been merged |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, as far as I can tell by looking on my phone, but I didn't actually check the paths etc.
I added an assert that checks the paths (otherwise we do perf on the rustc-perf repo instead if the path is mistyped thanks to how our discovery works 😅), so they are good @bors r=lnicola |
Bump `rustc-perf` checkout for metrics, replace webrender, diesel and ripgrep with newer versions The previous versions were removed hence the updates. The metric results for the changed ones are disconnected now as to not muddle things up. I think it's worth it for us to occasionally bump the `rustc-perf` checkout, as it allows us to include more interesting new targets. Notably, the newer diesel is 3 times as fast to analyze as the one of the current checkout.
@bors r- |
☀️ Try build successful - checks-actions |
Had to change CI to pick up the new files |
Bump `rustc-perf` checkout for metrics, replace webrender, diesel and ripgrep with newer versions The previous versions were removed hence the updates. The metric results for the changed ones are disconnected now as to not muddle things up. I think it's worth it for us to occasionally bump the `rustc-perf` checkout, as it allows us to include more interesting new targets. Notably, the newer diesel is 3 times as fast to analyze as the one of the current checkout.
@bors r+ |
☀️ Test successful - checks-actions |
rustc-perf
checkout for metrics, replace webrender, diesel and ripgrep with newer versionsrustc-perf
checkout for metrics, replace webrender, diesel and ripgrep with newer versions
The previous versions were removed hence the updates. The metric results for the changed ones are disconnected now as to not muddle things up.
I think it's worth it for us to occasionally bump the
rustc-perf
checkout, as it allows us to include more interesting new targets.Notably, the newer diesel is 3 times as fast to analyze as the one of the current checkout.