-
Notifications
You must be signed in to change notification settings - Fork 73
Add utility methods for job reconciliation #24
Add utility methods for job reconciliation #24
Conversation
@richardsliu Coverage/coveralls calculation does not seem correct (it says |
Thanks @richardsliu. Just updated so they are more readable now. |
LGTM Is there anyone working on service reconciliation logic? |
@gaocegege Yes someone from our team is putting together a PR (should be out soon). |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: richardsliu The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
This reverts commit 8a94e8f.
This PR adds utility methods for job reconciliation (part of #7). Note that part of the code is commented out for now until service reconciliation logic is in place.
This change is