Adds operators to Frequency #1473
Conversation
| @@ -9,4 +9,16 @@ class FrequencyTest { | |||
| assertEquals(100.milliseconds, 10.timesPerSecond.timeSpan) | |||
| assertEquals(10.hz, 100.milliseconds.hz) | |||
| } | |||
|
|
|||
| @Test | |||
| fun `test frequency operators work as expected`() { | |||
There was a problem hiding this comment.
This might not work, since on JS and Android IIRC this might not be valid. Let's wait for CI
Codecov ReportPatch coverage:
📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more Additional details and impacted files@@ Coverage Diff @@
## main #1473 +/- ##
==========================================
+ Coverage 51.58% 51.59% +0.01%
==========================================
Files 1680 1680
Lines 98906 98953 +47
Branches 13938 13944 +6
==========================================
+ Hits 51022 51059 +37
- Misses 44027 44029 +2
- Partials 3857 3865 +8
Flags with carried forward coverage won't be shown. Click here to find out more.
... and 28 files with indirect coverage changes Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report in Codecov by Sentry. |
|
LGTM. Thanks! 🎉 |
Based on TimeSpan