Skip to content
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

ci(benchmarks): update benchmark workflow for push on master #2891

Merged
merged 18 commits into from
Oct 3, 2024

Conversation

sw360cab
Copy link
Contributor

@sw360cab sw360cab commented Oct 3, 2024

Restructuring Benchmarks workflow in the following way

  • Reducing workflows for benchmarks to a single one (instead of two different ones)
  • Workflow is triggered only when PR is finally merged with master branch
  • There is a single flow of benchmarks performance to verify
  • Benchmarks do not impact PR
  • An Alert is automatically added as comment to each commit when regressions are introduced respect to previous results

@ajnavarro ajnavarro changed the title ci(benchmarks) Feat/benchmarks master push chore: ci(benchmarks) Feat/benchmarks master push Oct 3, 2024
Copy link

codecov bot commented Oct 3, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 60.95%. Comparing base (a2b4d4b) to head (0b8f66e).
Report is 2 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #2891   +/-   ##
=======================================
  Coverage   60.95%   60.95%           
=======================================
  Files         564      564           
  Lines       75273    75273           
=======================================
+ Hits        45885    45886    +1     
- Misses      26018    26019    +1     
+ Partials     3370     3368    -2     
Flag Coverage Δ
contribs/gnodev 61.46% <ø> (ø)
contribs/gnofaucet 15.31% <ø> (+0.85%) ⬆️
gno.land 67.92% <ø> (ø)
gnovm 65.77% <ø> (ø)
misc/genstd 80.54% <ø> (ø)
misc/logos 19.88% <ø> (-0.36%) ⬇️
tm2 62.11% <ø> (+0.04%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@thehowl thehowl changed the title chore: ci(benchmarks) Feat/benchmarks master push ci(benchmarks): update benchmark workflow for push on master Oct 3, 2024
@sw360cab sw360cab merged commit 41881ec into gnolang:master Oct 3, 2024
121 of 122 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants