Integrated: 1344: Support notifications operating on a repository mirror
Erik Joelsson
erikj at openjdk.java.net
Tue Feb 15 15:58:20 UTC 2022
On Mon, 14 Feb 2022 22:43:45 GMT, Erik Joelsson <erikj at openjdk.org> wrote:
> We would like to move some NotifierBot configurations to operate off of a mirror of a repo instead of the original repo. In order to support this I would like to add these two new configuration options in the issue notifier:
>
> 1. "originalrepository": Setting this implies that the repo the notifier is operating on is a mirror, and this original repo should be used for any links generated in notifications.
> 2. "repoonly": Similar to the existing "pronly", this limits what kind of notifications the notifier acts on. This is needed to be able to separate the PR event based notifications from the repo event based notifications. PRs aren't mirrored so those can't be moved to operate from the mirror.
>
> When implementing repoonly, I also changed the NotifyBot to only return WorkItems for either pr or repo if there are actually listeners registered for them respectively. This caused some seemingly unrelated tests to fail, as they were relying on the side effect of running the RepositoryWorkItem with nothing to do. To combat this, I've introduced a mock NullRepositoryListener for these tests.
This pull request has now been integrated.
Changeset: 5dbc91be
Author: Erik Joelsson <erikj at openjdk.org>
URL: https://git.openjdk.java.net/skara/commit/5dbc91be892ddf178a77749b3f1232f43b83d0c3
Stats: 156 lines in 7 files changed: 128 ins; 5 del; 23 mod
1344: Support notifications operating on a repository mirror
Reviewed-by: ihse, kcr
-------------
PR: https://git.openjdk.java.net/skara/pull/1285
More information about the skara-dev
mailing list