Add support for Codspeed performance testing#20
Conversation
tarcieri
left a comment
There was a problem hiding this comment.
Haven't heard of this before but it looks interesting
|
@bifurcation something else that might be interesting is |
| - uses: dtolnay/rust-toolchain@master | ||
| with: | ||
| toolchain: stable | ||
| - run: cargo install cargo-codspeed |
There was a problem hiding this comment.
This will compile cargo-codspeed on each CI run, which is really inefficient. It's better to use pre-compiled releases, e.g. see how we handle cargo-hack installation.
There was a problem hiding this comment.
Thanks, will try to get things working and then optimize :)
|
@tarcieri Sorry, should have marked this as draft! I'm just playing around with it at this point. I don't think the current rev even works. (Note that there are no real CI checks, since I broke the YAML file.) Probably best to revert until this is debugged. |
|
Opened #21 to revert (the GitHub UI for YAML parse failures is awkwardly deceptive, unfortunately) |
This reverts commit bcfd143.
No description provided.