Increase limit job about merge-gatekeeper HOT 6 CLOSED

@yuriydzobak Sorry for getting back to you late!
Understood, we will do further testing on our side. Merge Gatekeeper aims to be a drop-in solution without any extra configurations for most standard monorepo use cases, so we would like it to handle cases where there are so many jobs running at the same time. We will check and get back to you as soon as we can!

from merge-gatekeeper.

Hi @rytswd
could you help with it?

from merge-gatekeeper.

Hi @yuriydzobak, thanks for raising the issue! ☺️
We will certainly need to look at this more closely, as this is new to me at least. Let us dive deep and figure out what's possibly causing this - depending on that, we may be able to ensure Merge Gatekeeper works with any number of jobs, or we may introduce another field, with which you can configure based on your requirements 👍

from merge-gatekeeper.

Hi @yuriydzobak, thanks for raising the issue! ☺️
We will certainly need to look at this more closely, as this is new to me at least. Let us dive deep and figure out what's possibly causing this - depending on that, we may be able to ensure Merge Gatekeeper works with any number of jobs, or we may introduce another field, with which you can configure based on your requirements 👍

Would be nice if MG works with any number, because I have repos with 20-30 jobs, some repo has 70 jobs(matrix) and for example, users don't care about any limits in MG
for me, any solution would be great 😂

Thank you

from merge-gatekeeper.

Sorry for the delay in getting back to this, just a quick update on this.
We have found that the GitHub API we use indeed only gives us 30 results, and this can be increased to 100. However, this is more of a pagination handling by GitHub, and in order to fully support any number of jobs, we would need to adjust our API usages in a bit more complicated manner (i.e. ensure pagination is taken into account). We think we have an implementation idea to fix this, but please bear with us until we work out the solution with proper tests in place 🙏

from merge-gatekeeper.

Given the above fix #48 is in, this should now be handled without any extra configurations. Merge Gatekeeper will simply follow the pagination to ensure we get the full list of jobs. Thanks to @Jrc356 for contributing a fix! 👍

from merge-gatekeeper.