Skip to content

Conversation

edenhaus
Copy link
Member

No description provided.

@Copilot Copilot AI review requested due to automatic review settings June 19, 2025 20:36
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR updates the CI workflow to run benchmark jobs immediately after the tests by adjusting job dependencies.

  • Replaces multiple build and packaging dependencies with a single tests dependency for the benchmark job.
Comments suppressed due to low confidence (1)

.github/workflows/ci.yml:339

  • Ensure that the tests job includes all build and packaging steps previously provided by build-test-native, build-test-musl, and sdist; otherwise, the benchmark job may run against missing or outdated artifacts.
      - tests

Copy link

codecov bot commented Jun 19, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 93.77%. Comparing base (3aea0c6) to head (c87eb60).

✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff           @@
##              dev    #1035   +/-   ##
=======================================
  Coverage   93.77%   93.77%           
=======================================
  Files         129      129           
  Lines        5014     5014           
  Branches      327      327           
=======================================
  Hits         4702     4702           
  Misses        249      249           
  Partials       63       63           

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

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copy link

codspeed-hq bot commented Jun 19, 2025

CodSpeed Performance Report

Merging #1035 will not alter performance

Comparing ci-benchmark-faster (c87eb60) with dev (3aea0c6)

Summary

✅ 6 untouched benchmarks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant