RFR: 2164: Add git notes notifier

Erik Duveblad ehelin at openjdk.org
Tue Jan 30 16:17:12 UTC 2024


On Mon, 29 Jan 2024 15:05:36 GMT, Erik Joelsson <erikj at openjdk.org> wrote:

> Could you add one of these to a test in NotifierBotFactoryTest? It's not complicated, but I like have example configurations in those tests as they also kind of serve as documentation.

Sure, will do!

> forge/src/main/java/org/openjdk/skara/forge/github/GitHubRepository.java line 587:
> 
>> 585:         var url = URI.create(o.get("html_url").asString());
>> 586:         var webUrl = gitHubHost.getWebURI(url.getPath());
>> 587:         return Optional.of(new HostedCommit(metadata, diffs, url, webUrl));
> 
> Should we provide an implementation for GitLab as well?

No I don't think so as we currently don't have a need to rewrite any GitLab URLs. We rewrite `github.com` to `git.openjdk.org` so that links stored in OpenJDK infrastructure (like JBS, mailing lists, etc.) aren't tried to GitHub (if we ever change forge then we can implement a re-direct service). For GitLab we currently don't have that need.

-------------

PR Comment: https://git.openjdk.org/skara/pull/1608#issuecomment-1917090262
PR Review Comment: https://git.openjdk.org/skara/pull/1608#discussion_r1471392150


More information about the skara-dev mailing list