updated!

that's a thousand times better, thanks! updated

didn't know that, updated

updated

good catch, I'm also adding that to the updateCommitStatus that's also logging this info

The build is currently failing due to a missing build artefact. I couldn't properly rebuild the old versions, so we're just upgrading as needed to ...

since we're doing calculations here, we should wrap this in a if (LOGGER.isTraceEnabled()) { ... }. Most of the time the processing is hitting a ge...

We shouldn't do this forever, just in case something changes in how data is returned. I'd just say add a condition like page < 50, as a PR shouldn'...

Objects.isNull isn't meant to be used in conditionals like this, but instead as a predicate in streams for easier calling. In cases like this, you ...

Is there a reason to use AutoValue here? We've been trying to move to records so that we can remove AutoValue eventually

See the following guide for how to do config mappings, as we should use that for this sort of object: https://quarkus.io/guides/config-mappings

@malowe Now that I'm finally using the proper API, this should work once we add the correct permissions to the installation and configure the bot I...

Nice to hear that!...

Moving it back to draft....