-
Notifications
You must be signed in to change notification settings - Fork 9
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
Decouple statuses and versions and add tests #137
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #137 +/- ##
==========================================
+ Coverage 83.45% 84.85% +1.40%
==========================================
Files 29 29
Lines 1408 1459 +51
==========================================
+ Hits 1175 1238 +63
+ Misses 233 221 -12
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Linting is not passing though.
7e0c534
to
b8e2e70
Compare
Done
Rationale
The current functionality requires you to provide a status query for every version. For the overhaul we want to be able to query for these independent of one another such that you can query for multiple versions and one status, for example.
QA
Issue / Card
Fixes https://warthogs.atlassian.net/browse/WD-8817