-
Notifications
You must be signed in to change notification settings - Fork 29
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
Replace String period start and end with ZonedDateTime and add new validator for ZonedDateTime periods. #538
Replace String period start and end with ZonedDateTime and add new validator for ZonedDateTime periods. #538
Conversation
…idation class for single and multi measure evaluation. Ensure all existing tests pass. No new tests yet.
Formatting check succeeded! |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #538 +/- ##
============================================
+ Coverage 72.56% 72.59% +0.03%
Complexity 19 19
============================================
Files 361 362 +1
Lines 18562 18597 +35
Branches 2869 2874 +5
============================================
+ Hits 13469 13501 +32
- Misses 3733 3734 +1
- Partials 1360 1362 +2 ☔ View full report in Codecov by Sentry. |
…ep some of the new unit test code for better assertions.
…count is still 0.
…timezone-for-measurement-period-3
…sts to String dates. Make a new test with period tests and move tests there. Clean up junk.
Quality Gate passedIssues Measures |
Closes #528