Haaaa, everyone, I am this example :)<br>
Sorry I haven't checked these mails cc to me for a long time.<br>
<br>
Junio C Hamano <<a href="mailto:gitster@pobox.com">gitster@pobox.com</a>> 于2021年4月17日周六 上午3:50写道:<br>
><br>
> Jonathan Nieder <<a href="mailto:jrnieder@gmail.com">jrnieder@gmail.com</a>> writes:<br>
><br>
> > 3. Do you think patchwork goes in a direction that is likely to help<br>
> > with these?<br>
><br>
> So here is a real-life example.<br>
><br>
> Let's say somebody is looking at a "gentle ping" [*1*]<br>
><br>
> znh> The patch seems to have fallen into the crack.<br>
> zhn> Jeff and Junio, willing to help?<br>
><br>
> How would we figure out what happened to the patch today without<br>
> visiting patchwork would be:<br>
><br>
> 1. Visit the message at <a href="http://lore.kernel.org/git/" target="_blank">lore.kernel.org/git/</a> [*1*]<br>
><br>
> 2. Notice that it is a response to a message, and click the link to<br>
> be taken to [*2*]<br>
><br>
> 3. Notice that nobody commented on the patch.<br>
><br>
> 4. Type "f:zhening ref-filter" to the search box and search, with<br>
> suspicion that this was an updated version of something.<br>
><br>
> 5. Click one of them in the result [*3*]<br>
><br>
<br>
> I can see #3 would immediately become obvious, and I hope #4-#5<br>
> would become unnecessary.<br>
><br>
<br>
Here are my thoughts:<br>
<br>
For the reviewers like Junio, after missing a new patch iteration, need to<br>
review the past history to find the correct patch and related comments<br>
from other reviewers. Just like I once read a github blog saying that<br>
"patch" is also a special object in git. I would like to have a "new" tool<br>
which can link multiple related patches and comments.<br>
<br>
1. Coder need Reviewers' help.<br>
2. This new tool will obtained multiple different patch versions automatically<br>
or coder provided those pathes versions links.<br>
3. This tool will analyze the differences between multiple versions, get all<br>
the reviewers comments and coder comments related to the "patch stream",<br>
organize it into "patch graph".<br>
4. The tool will notify the reviewer(by email or something else) and show the links<br>
and patch graph or patch range-diff.And It can visualize the entire patch process,<br>
It’s best that comments from different people can be displayed on one page.<div><br>
In order to be more accurate, I made a picture [*1*].<br>
<br>
Using this new tool, reviewers can choose to see or not see the range-diff<br>
and diff in multiple different patch versions, Instead of the range-diff<br>
automatically sent by GGG. When my second patch processing was greatly<br>
changed from the previous one, I have to rebuild a new branch and create a new<br>
PR, this is my pain point.<br>
<br>
> Anything else?<br>
><br>
> At steps #6 and #7, there is human judgment involved that may not be<br>
> automatable, but would there be some mechanism to make it easy to<br>
> help these steps if the user visits patchwork (instead of staying<br>
> in my newsreader or web interface to the lore archive)?<br>
><br>
> I am of course not expecting to automate step #9 ;-) It would be<br>
> nice though.<br>
><br>
> Thanks.<br>
><br>
><br>
> [References]<br>
><br>
> *1* <a href="https://lore.kernel.org/git/CAOLTT8Tis5Yjg8UR0c-i0BnqiFQvLXvDgxUQJ-WcP6jjQPu9cQ@mail.gmail.com/" target="_blank">https://lore.kernel.org/git/<wbr>CAOLTT8Tis5Yjg8UR0c-<wbr>i0BnqiFQvLXvDgxUQJ-<wbr>WcP6jjQPu9cQ@mail.gmail.com/</a><br>
><br>
> *2* <a href="https://lore.kernel.org/git/pull.928.git.1617975348494.gitgitgadget@gmail.com/" target="_blank">https://lore.kernel.org/git/<wbr>pull.928.git.1617975348494.<wbr>gitgitgadget@gmail.com/</a><br>
><br>
> *3* <a href="https://lore.kernel.org/git/pull.927.v2.git.1617809209164.gitgitgadget@gmail.com/" target="_blank">https://lore.kernel.org/git/<wbr>pull.927.v2.git.1617809209164.<wbr>gitgitgadget@gmail.com/</a><br>
<br>Thanks!</div><div><br></div><div>
*1* <a href="https://github.com/adlternative/git/blob/pic/git-patch-pain-point-solve-idea.png">https://github.com/adlternative/git/blob/pic/git-patch-pain-point-solve-idea.png</a><br>
</div><div><br></div><div>--</div><div>ZheNing Hu</div>