SeongJae Park <[email protected]> writes: > For review of patches that revisioned multiple times, patch changelogs > are very useful. Adding actual links to the previous versions can > further help the review. Using such links, reviewers can double check > the changelog by themselves, and find previous discussions. Nowadays > having such links (e.g., lore.kernel.org archive links) is easy and > reliable. Suggest adding such links if available. > > Signed-off-by: SeongJae Park <[email protected]> > --- > Documentation/process/submitting-patches.rst | 6 +++++- > 1 file changed, 5 insertions(+), 1 deletion(-) > > diff --git a/Documentation/process/submitting-patches.rst > b/Documentation/process/submitting-patches.rst > index 9a509f1a6873..e69d19ad658f 100644 > --- a/Documentation/process/submitting-patches.rst > +++ b/Documentation/process/submitting-patches.rst > @@ -805,7 +805,8 @@ not part of the changelog which gets committed to the git > tree. It is > additional information for the reviewers. If it's placed above the > commit tags, it needs manual interaction to remove it. If it is below > the separator line, it gets automatically stripped off when applying the > -patch:: > +patch. If available, adding links to previous versions of the patch (e.g., > +lore.kernel.org archive link) is recommended to help reviewers:: > > <commit message> > ... > @@ -814,6 +815,9 @@ patch:: > V2 -> V3: Removed redundant helper function > V1 -> V2: Cleaned up coding style and addressed review comments > > + v2: https://lore.kernel.org/bar > + v1: https://lore.kernel.org/foo > +
Applied, thanks. jon
