lkml.org 
[lkml]   [2023]   [Jan]   [20]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    SubjectRe: [PATCH v4 0/3] checkpatch.pl: warn about discouraged tags and missing Link: tags
    From
    Date
    On Fri, 2023-01-20 at 13:35 +0100, Thorsten Leemhuis wrote:
    > Hi, please consider the following checkpatch.pl patches for review.
    >
    > The first two changes make checkpatch.pl check for a few mistakes wrt to
    > links to bug reports Linus recently complained about a few times.
    > Avoiding those is also important for my regression tracking efforts a
    > lot, as the automated tracking performed by regzbot relies on the proper
    > usage of the Link: tag.
    >
    > The third patch fixes a few small oddities noticed in existing code
    > during review of the two changes.

    Hey Andrew. Please forward this patch series upstream.

    Thanks, Joe

    >
    > Ciao, Thorsten
    > ---
    > v3->v4:
    > - address review feedback from Joe (do not allow leading spaces in
    > matches, check if Link: is actually followed by a URL in one go,
    > and use m{} for matching https://) (Thorsten)
    > - catch Reported-and-tested-by as well (Thorsten)
    >
    > v2->v3:
    > - address review feedback from Joe (grammer fixes, use of $herecurr, use
    > of --no-merges in commit log quote) (Thorsten)
    > - warn when people try to add version information to the commit log
    > using code suggested by Joe (Thorsten)
    > - add a patch to make existing code in a nearby area use $herecurr where
    > it should, which Joe noticed during review
    >
    > v1->v2:
    > - add commit messages explaining the reasoning (Thorsten)
    > - approach the usage of wrong tags with an allow list as suggested by
    > Joe (Thorsten)
    >
    > Kai Wasserbäch (2):
    > checkpatch: warn when unknown tags are used for links
    > checkpatch: warn when Reported-by: is not followed by Link:
    >
    > Thorsten Leemhuis (1):
    > checkpatch: use proper way for show problematic line
    >
    > scripts/checkpatch.pl | 34 +++++++++++++++++++++++++++++-----
    > 1 file changed, 29 insertions(+), 5 deletions(-)
    >
    >
    > base-commit: 5dc4c995db9eb45f6373a956eb1f69460e69e6d4

    \
     
     \ /
      Last update: 2023-03-26 23:49    [W:4.100 / U:1.200 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site