RFR: 2401: Link to Webrev should be part of the Reviewing section
Erik Joelsson
erikj at openjdk.org
Wed Oct 30 12:43:58 UTC 2024
On Wed, 30 Oct 2024 11:40:51 GMT, Magnus Ihse Bursie <ihse at openjdk.org> wrote:
> When we added the link to the webrev comment in the PR body, for some reason it got its own header `### Webrev`. But Webrevs are just another way of reviewing, so it should be put under the `### Reviewing` header.
>
> That is, the reviewing html code should look something like this:
>
> ...
> <details><summary>Using diff file</summary>
>
> Download this PR as a diff file: \
> <a href="https://git.openjdk.org/jdk/pull/XXX.diff">[https://git.openjdk.org/jdk/pull/XXX.diff</a](https://git.openjdk.org/jdk/pull/XXX.diff%3C/a)>
>
> </details>
> <details><summary>Using Webrev</summary>
>
> [Link to Webrev Comment](https://git.openjdk.org/jdk/pull/XXX#issuecomment-YYY)
>
> </details>
Marked as reviewed by erikj (Lead).
-------------
PR Review: https://git.openjdk.org/skara/pull/1695#pullrequestreview-2404742135
More information about the skara-dev
mailing list