Rebased to master. Slapping a pullrequest on this.
The branch includes release notes, but no unit tests. My concern with unit tests is that we'd have to create /a lot/ of unit test code to have even a modest amount of coverage for the hold targeter. Thoughts welcome on that, of course.
At minimum, I believe this code is ready for human testing.
Rebased to master. Slapping a pullrequest on this.
The branch includes release notes, but no unit tests. My concern with unit tests is that we'd have to create /a lot/ of unit test code to have even a modest amount of coverage for the hold targeter. Thoughts welcome on that, of course.
At minimum, I believe this code is ready for human testing.