RFR: 1271: Webrev generation should be optional

Erik Joelsson erikj at openjdk.org
Thu Feb 16 19:54:11 UTC 2023


On Thu, 16 Feb 2023 00:05:55 GMT, Zhao Song <zsong 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.

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

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


More information about the skara-dev mailing list