RFR: 1271: Webrev generation should be optional

Zhao Song zsong at openjdk.org
Thu Feb 16 21:51:34 UTC 2023


On Thu, 16 Feb 2023 19:52:01 GMT, Erik Joelsson <erikj at openjdk.org> wrote:

> > Above `ArchiveMessages#composeConversationFooter`, there is a comment says `// When changing this, ensure that the PR pattern in the notifier still matches`. I tried to figure out where the PR pattern is, but I couldn't find it.
> 
> I believe it's the logic that parses email from the email list server and extracts the PR URL from the verify first email in the each thread. You are likely safe here as you aren't changing the line that contains the PR URL, but better be safe.

Just see this comment. I will check it again.

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

PR: https://git.openjdk.org/skara/pull/1475


More information about the skara-dev mailing list