Skip to content

Publish benchmarks in job summary #10

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

Merged
merged 1 commit into from
Mar 16, 2023
Merged

Conversation

bennettgoble
Copy link
Member

@bennettgoble bennettgoble commented Mar 16, 2023

Only store benchmark results from commits to main. This will allow actual comparisons to PRs to happen.

@codecov
Copy link

codecov bot commented Mar 16, 2023

Codecov Report

Merging #10 (a4b0a7a) into main (8fdaf68) will not change coverage.
The diff coverage is n/a.

❗ Current head a4b0a7a differs from pull request most recent head 539df6d. Consider uploading reports for the commit 539df6d to get more accurate results

@@           Coverage Diff           @@
##             main      #10   +/-   ##
=======================================
  Coverage   89.83%   89.83%           
=======================================
  Files           6        6           
  Lines         797      797           
=======================================
  Hits          716      716           
  Misses         81       81           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

Only store benchmark results from commits to main. This will allow
actual comparisons to PRs to happen.
@bennettgoble bennettgoble force-pushed the signal/bench-summary branch from a4b0a7a to 539df6d Compare March 16, 2023 19:48
@bennettgoble bennettgoble merged commit 8f5faba into main Mar 16, 2023
@bennettgoble bennettgoble deleted the signal/bench-summary branch March 16, 2023 19:51
@github-actions github-actions bot locked and limited conversation to collaborators Mar 16, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant