Missing PRs randomly in release notes

[michaelrosett]

Missing PRs randomly in release notes

[michaelrosett]

It would be difficult to reproduce this as it has to do with the number of PRs and the blackbox that is the GitHub API - it is unclear what underlying factors cause unpredictable response ordering.

I have verified the new release candidate against tags in our private repo where we saw the issue. Thank you!

[mikepenz]

Thank you very much for verifying the changes!

[michaelrosett]

According to https://docs.github.com/en/rest/pulls/pulls?apiVersion=2022-11-28#list-pull-requests, sort only supports the values created, updated, popularity, long-running

[mikepenz]

Thank you very much for raising this ticket.

In the past merged did work, and we have a good amount of tests to check if PRs are retrieved. That said I wonder if because it is not on the docs (anymore(?)) it can have undefined behaviour.

That said, there is a big problem going with updated as it will return many more items (basically any PR you make any change to, even if it's just updating the description) will be sorted ahead.

There are certainly additional implications we need to be careful about, for example fetchedEnough would very likely need to also be adjusted to only consider the update date.

Is there any way you can reproduce this consistently on a open source repo? That would immensely help to setup a test case for it.

[mikepenz]

In the process of tagging a release candidate v4.2.0-rc01 which you will be able to use