Re: [PATCH v2 1/2] README: restructure with role-based documentation and guidelines

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Sun, Aug 10, 2025 at 08:44:58AM -0700, Joe Perches wrote:
On Sat, 2025-08-09 at 19:40 -0400, Sasha Levin wrote:
Reorganize README to provide targeted documentation paths for different
user roles including developers, researchers, security experts,
maintainers, and AI coding assistants. Add quick start section and
essential docs links.

Include proper attribution requirements for AI-assisted contributions
using Assisted-by tags with agent details and tools used.

Nicely done.

Thanks Joe!

Perhaps the 'Assisted-by:' tag should not be limited to AI
assistance but could also be used when accepted notes were
given on any revised patch submission.

The suggestions from the previous patches around expanding this to be a
list of tools rather than just "AI" made sense, this is the example I
gave in the cover letter:

	Assisted-by: Claude-claude-3-opus-20240229 checkpatch

I find something like that useful because it tells me from the get-go
that the submitter ran checkpatch on it (without having to spend a line
in the commit message saying the same).

I'm not sure about mixing human feedback into this, it might be
difficult to interpert it later.

It might work more naturally as an extension of Reviewed-by?

	Reviewed-by: Developer A <a@b.c> # Improved the XYZ algorithm

Oh, and maybe a checkpatch update like this?
---
scripts/checkpatch.pl | 1 +
1 file changed, 1 insertion(+)

diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl
index e722dd6fa8ef3..d17661141da79 100755
--- a/scripts/checkpatch.pl
+++ b/scripts/checkpatch.pl
@@ -641,6 +641,7 @@ our $signature_tags = qr{(?xi:
	Reviewed-by:|
	Reported-by:|
	Suggested-by:|
+	Assisted-by:|
	To:|
	Cc:
)};

Yup, makes sense! I'll start including checkpatch updates going forward.

--
Thanks,
Sasha




[Index of Archives]     [Kernel Newbies]     [Security]     [Netfilter]     [Bugtraq]     [Linux FS]     [Yosemite Forum]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Device Mapper]     [Linux Resources]

  Powered by Linux