Skip to content

Add script to extract the authors from git log#4990

Open
afbjorklund wants to merge 2 commits into
lima-vm:masterfrom
afbjorklund:authors
Open

Add script to extract the authors from git log#4990
afbjorklund wants to merge 2 commits into
lima-vm:masterfrom
afbjorklund:authors

Conversation

@afbjorklund
Copy link
Copy Markdown
Member

@afbjorklund afbjorklund commented May 16, 2026

As per the AI discussion about authors, added a script to extract "The Lima Authors"

Lima
Copyright The Lima Authors.

make AUTHORS

(inspired by containerd AUTHORS)

The humans are still in the lead (I didn't add you to the mailmap, so you get two entries)

  2808	Akihiro Suda
  1286	Jan Dubois
   709	dependabot[bot] 
   337	Anders F Björklund
   185	Norio Nomura

The file has email, too.

But didn't list them here.

Signed-off-by: Anders F Björklund <anders.f.bjorklund@gmail.com>
@afbjorklund
Copy link
Copy Markdown
Member Author

afbjorklund commented May 16, 2026

Besides dependabot, there's also:

     1	Claude Opus 4.5
     2	Claude Opus 4.6
     1	Claude Sonnet 4.6
     1	Copilot Autofix powered by AI
     1	copilot-swe-agent[bot]
     3	StepSecurity Bot

@jandubois
Copy link
Copy Markdown
Member

jandubois commented May 18, 2026

Besides dependabot, there's also:

I think we should filter them out. For now matching [bot], <bot@, or <noreply@ catches them all: grep -vE '\[bot\]|<(bot|noreply)@'.

Is the plan to include the list in the repo itself? It will be a bit of a maintenance problem unless we lazily update it with e.g. a scheduled action.

@afbjorklund
Copy link
Copy Markdown
Member Author

Besides dependabot, there's also:

I think we should filter them out. For now matching [bot], <bot@, or <noreply@ catches them all: grep -vE '\[bot\]|<(bot|noreply)@'.

They will not be reported at all, when using Assisted-by instead of Co-Authored-by (or Author)

I was just curious about the current situation in the repository, and about the copyright vs author

Is the plan to include the list in the repo itself? It will be a bit of a maintenance problem unless we lazily update it with e.g. a scheduled action.

No, only the script. The list is already included in the repo, just "hidden" in the git log information

If we want to export something, we will need to use the Signed-Off-by and also include vendor...

@afbjorklund
Copy link
Copy Markdown
Member Author

afbjorklund commented May 18, 2026

Also: dependabot doesn't currently add any code directly, only code by others and indirectly.
i.e. the code is in the commits or versions being bumped, not in the line with the reference

So it is not really a part of the discussion about the AI agents, it is more of a tool (like sed)
Despite being currently listed as an "author", which is just a quirk of how git commits work?

Signed-off-by: dependabot[bot] <support@github.com>


EDIT: Seeing the changes introduced by dependabot would be an interesting tool by itself...

Like dumping "vendor" before and after commit, or try to find the external commits if any?

EDIT: It does include the "Changelog" and the "Commits" in the PR itself, though:

https://github.com/lima-vm/lima/pulls?q=is%3Apr+author%3Aapp%2Fdependabot+

So maybe it already has a local version as well, didn't check.

Signed-off-by: Anders F Björklund <anders.f.bjorklund@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants