Skip to content

Conversation

@kozlov721
Copy link
Collaborator

Purpose

More meaningful error when an empty split definitions are passed to make_splits

Specification

None / not applicable

Dependencies & Potential Impact

None / not applicable

Deployment Plan

None / not applicable

Testing & Validation

None / not applicable

@kozlov721 kozlov721 requested a review from a team as a code owner May 19, 2025 08:59
@kozlov721 kozlov721 requested review from conorsim, klemen1999 and tersekmatija and removed request for a team May 19, 2025 08:59
@github-actions github-actions bot added the fix Fixing a bug label May 19, 2025
@codecov
Copy link

codecov bot commented May 19, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 95.55%. Comparing base (f9b018a) to head (65d0972).
Report is 1 commits behind head on main.

✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff            @@
##           main     #326       +/-   ##
=========================================
+ Coverage      0   95.55%   +95.55%     
=========================================
  Files         0      102      +102     
  Lines         0     6071     +6071     
=========================================
+ Hits          0     5801     +5801     
- Misses        0      270      +270     

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

@kozlov721 kozlov721 merged commit f2cf992 into main May 19, 2025
16 checks passed
@kozlov721 kozlov721 deleted the fix/empty-split-definitions branch May 19, 2025 09:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fix Fixing a bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants