Is webrev generation still relevant?
Philip Race
philip.race at oracle.com
Fri Jan 19 17:24:49 UTC 2024
+1 to keep webrevs.
I look at them regularly for all the reasons given so far and more,
for example I prefer to download the patch webrev generates rather than
use other mechanisms to apply a fix to test in my local repo.
When there are many files in a patch github just breaks down in many cases.
webrev also gives me more ways to view the incremental and full patches.
It is also clear, less busy and straightforward and I actually trust
them more.
If it were not for the fact comments have to be given in the PR on
github I would
probably only use webrevs and ignore github.
-phil.
On 1/18/24 1:04 AM, Magnus Ihse Bursie wrote:
> At the onset of Project Skara, one goal was to keep backwards
> compatibility with developers' workflows. For this, a Skara bot was
> created which generates webrevs, as closely aligned to the original
> ksh webrev script as possible.
>
> Now I believe all developers are well into the Skara/GitHub way of
> doing things, and I have not heard someone refer to webrevs in a long
> time. So my first question is:
>
> * Is it still relevant to continue let the Skara bots generate webrevs?
>
> I personally have only used webrevs on a few occasions the last years,
> and those have all been when the GitHub diff viewer was inadequate.
> For instance, the webrev bot uses a more aggressive method of letting
> git match files that have been simultaneously moved and edited, and
> the Frames view align code side-by-side which is sometimes much more
> helpful than the line-by-line view in GitHub. So, my second question is:
>
> * Should we keep the idea of a bot that generates diff pages, but
> instead of mimicking the old webrev script, tailor it to cover up for
> those use cases where GitHub falls short?
>
> I'm not suggesting we should immediately turn of the webrev bot, so if
> you still like and use it, there is no cause for panic. I'm just
> trying to get a sense of how people feel about the future for webrevs.
>
> /Magnus
>
More information about the jdk-dev
mailing list