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
Update benchmark crates and workflow #3027
Update benchmark crates and workflow #3027
Conversation
this doesn't comment on PR directly, still benchmark-action/github-action-benchmark has open issues about it
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!
Can we delete the previous data? I don't think we should keep too much data around because of differences between GitHub provided runners. Also, our previous data wasn't really accurate either
I think we have to delete the data from the |
Description
Updates the benchmark setup to the latest
js-framework-benchmark
setup.Also separates the posting of comments and the update to the gh-pages branch to run the benchmarking with less priviledges.
When reviewing, pay most attention to .github/workflows/post-benchmark.yml as that workflow is not run as part of the PR (on purpose as it has elevated privileges).
Checklist