If this gets merged, what would it take to get a new release? I would like to updatehttps://github.com/github-changelog-generator/docker-github-changelog-generator too.

@devinlyons I can see that this would fix that issue, can you figure out a test case that could go in the repo, too?

@olleolleolle, my Ruby skills are a bit weak. But I did look through the tests to see if I could find an example to work from. It looks like thefetch_comments_async function is only ever mocked in the tests. I guess this makes sense given that it does a lot of IO. So, I'm not sure how to test this. The use case is something likeit does not throw an exception when a page returns no comments. Do you have any guidance on this?

I took a look at the yield which this conditional is protecting from.

Was curious if the#iterate_pages could be made so that it didn't yield if it got a nillast_response.data (which is used inyield(last_response.data)) in#iterate_pages.

# @yield [Sawyer::Resource] An OctoKit-provided response (which can be empty)

I also couldn't see a quick way to test-drive that thing.

(This morning it dawned on me, "perhaps the data is never there, after some API change?")

@olleolleolle, I updated the function so it should never yield null.