]> git.ipfire.org Git - thirdparty/squid.git/commit
CI: Support customizing CONTRIBUTORS in new contributor PRs (#2128) auto master
authorEduard Bagdasaryan <eduard.bagdasaryan@measurement-factory.com>
Tue, 29 Jul 2025 12:26:28 +0000 (12:26 +0000)
committerSquid Anubis <squid-anubis@squid-cache.org>
Thu, 31 Jul 2025 03:59:51 +0000 (03:59 +0000)
commitc21da2bf83a9fc3e60c189f547a850dc2aa041de
tree80fff69bd3849988ed6613c5b3210e59cb13ab1b
parent0d7760d2fcf304357d980c5f2dc531a8b62d400c
CI: Support customizing CONTRIBUTORS in new contributor PRs (#2128)

When our source-maintenance.sh recognizes a magic phrase in a commit
message, it treats that commit as authoritative for CONTRIBUTORS file.
That feature allows us to customize CONTRIBUTORS by adding a magic
phrase to the commit message, preventing unwanted automated
collectAuthors() actions for earlier commits. Such commits work well
enough when CONTRIBUTORS is customized by an already known Squid
developer in a dedicated PR.

The same feature cannot work for PRs created by new developers because
when quick.yaml tests a PR, the first commit visible to collectAuthors()
is not a PR branch commit that the author could, with some considerable
trouble, customize, but a so called GitHub PR "merge commit" that GitHub
automatically creates with PR creator as the author. That merge commit
message does not contain PR description and never has our magic phrase.
That merge commit author details are wrong when PR creators want to use
information that differs from their public GitHub account info.

With this change, collectAuthors() consults PR description (when testing
PRs), allowing CONTRIBUTORS customization without any extra efforts:
Adding the magic phrase to PR description has to be done anyway.
.github/workflows/quick.yaml
scripts/source-maintenance.sh