Publish only results included in current invocation to checks api - fixes #198

[mrginglymus]

This, I believe, should fix the problem described in #198. However, I'm at a bit of a loss as to how best to test it, short of mocking a checks publisher and inspecting calls to that.

fixes #198

[timja]

This, I believe, should fix the problem described in #198. However, I'm at a bit of a loss as to how best to test it, short of mocking a checks publisher and inspecting calls to that.

I think just manually testing this bit is fine. otherwise yeah verifying the second call only contains the second results and not the combination of both would be the first to go.

[mrginglymus]

@timja apologies for the delay here. I've finally had a chance to test this on our jenkins and it does the job 👍

[mrginglymus]

Thanks for merging this @timja; do you have a rough idea of when you might next cut a release?

[timja]

Thanks for merging this @timja; do you have a rough idea of when you might next cut a release?

thanks for the nudge, I've triggered it. Should be available in ~15 mins or so

[mrginglymus]

Thanks!