All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jonathan Corbet <corbet@lwn.net>
To: "Ignacio Peña" <ignacio.pena87@gmail.com>
Cc: linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org,
	Ignacio Pena <ignacio.pena87@gmail.com>
Subject: Re: [PATCH] Documentation: Add patch-validator to dev-tools
Date: Thu, 17 Jul 2025 07:38:41 -0600	[thread overview]
Message-ID: <874ivbc4ym.fsf@trenco.lwn.net> (raw)
In-Reply-To: <20250717074745.8333-1-ignacio.pena87@gmail.com>

Ignacio Peña <ignacio.pena87@gmail.com> writes:

> From: Ignacio Pena <ignacio.pena87@gmail.com>
>
> Add documentation for patch-validator, a comprehensive tool that helps
> kernel contributors validate their patches before submission. This tool
> catches common mistakes that frequently lead to patch rejections,
> improving the quality of submissions and reducing maintainer workload.
>
> The validator performs 21+ automated checks including:
> - Patch format validation (subject line, changelog placement)
> - DCO compliance and licensing verification  
> - checkpatch.pl integration with enhanced reporting
> - Build impact analysis and testing recommendations
> - Common novice pattern detection
> - Git configuration validation
>
> Also includes companion tools for finding first contributions,
> testing patches safely, and validating patch series.
>
> Link: https://github.com/ipenas-cl/kernel-patch-validator
> Signed-off-by: Ignacio Pena <ignacio.pena87@gmail.com>
> ---
>  Documentation/dev-tools/index.rst           |   1 +
>  Documentation/dev-tools/patch-validator.rst | 287 +++++++++++++++++++
>  2 files changed, 288 insertions(+)
>  create mode 100644 Documentation/dev-tools/patch-validator.rst

Interesting ... overall, we don't generally have detailed documentation
for out-of-tree utilities, though there isn't necessarily any reason why
we couldn't.  But I'm curious as to why you haven't submitted the tool
itself?

Thanks,

jon

  reply	other threads:[~2025-07-17 13:38 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-07-17  7:47 [PATCH] Documentation: Add patch-validator to dev-tools Ignacio Peña
2025-07-17 13:38 ` Jonathan Corbet [this message]
     [not found] <87wmjy92r6.fsf@trenco.lwn.net>
2025-07-17 16:29 ` Ignacio Peña
2025-07-17 19:41   ` Jonathan Corbet

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=874ivbc4ym.fsf@trenco.lwn.net \
    --to=corbet@lwn.net \
    --cc=ignacio.pena87@gmail.com \
    --cc=linux-doc@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.