Skip to content

Conversation

@ales-erjavec
Copy link
Contributor

Issue
Description of changes

Specify time unit when converting from numeric to time.

Includes
  • Code changes
  • Tests
  • Documentation

@codecov
Copy link

codecov bot commented Nov 19, 2025

Codecov Report

❌ Patch coverage is 94.47853% with 9 lines in your changes missing coverage. Please review.
✅ Project coverage is 88.92%. Comparing base (c03a644) to head (02f6ac2).
⚠️ Report is 20 commits behind head on master.

❌ Your patch status has failed because the patch coverage (94.47%) is below the target coverage (95.00%). You can increase the patch coverage or adjust the target coverage.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #7192      +/-   ##
==========================================
- Coverage   88.92%   88.92%   -0.01%     
==========================================
  Files         335      335              
  Lines       74073    74191     +118     
==========================================
+ Hits        65872    65972     +100     
- Misses       8201     8219      +18     
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@ales-erjavec ales-erjavec force-pushed the oweditdomain-reinterpret-numeric-as-time branch from 7b18aa8 to 02f6ac2 Compare November 19, 2025 11:07
@ajdapretnar
Copy link
Contributor

Thank you so much! This is epic!

@ajdapretnar ajdapretnar merged commit cd252e3 into biolab:master Nov 21, 2025
24 of 30 checks passed
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.

Edit Domain: Converting to time fails on years < 1677

2 participants