This is an automated email from the ASF dual-hosted git repository.
noob-se7en pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/pinot.git
The following commit(s) were added to refs/heads/master by this push:
new f68057ad330 [#18688] Remove AI co-author trailers from commit history
(#18689)
f68057ad330 is described below
commit f68057ad330bf0cee22a8174980aa12c0d2b1dde
Author: Minwoo Jung <[email protected]>
AuthorDate: Fri Jun 5 18:41:18 2026 +0900
[#18688] Remove AI co-author trailers from commit history (#18689)
* [#18688] Remove AI co-author trailers from commit history guidance
Add a "Commit messages" section to CLAUDE.md instructing AI tools
not to include Co-authored-by trailers. These trailers propagate
into squash-merge commits, making project history appear AI-authored.
* [#18688] Add settings.json to disable Co-authored-by trailer
Claude Code respects includeCoAuthoredBy in .claude/settings.json.
Setting it to false ensures the trailer is never appended, regardless
of individual user configuration.
* [#18688] Add commit message rule to AGENTS.md for non-Claude AI tools
Per review feedback from noob-se7en: AGENTS.md is read by Copilot and
other AI tools that don't consume .claude/settings.json. Adding the
same Co-authored-by rule here ensures coverage across all AI tools.
* [#18688] Use attribution setting instead of deprecated includeCoAuthoredBy
includeCoAuthoredBy is deprecated. Replace with the current attribution
setting which controls both commit and PR trailers.
---
.claude/settings.json | 7 +++++++
AGENTS.md | 5 +++++
CLAUDE.md | 5 +++++
3 files changed, 17 insertions(+)
diff --git a/.claude/settings.json b/.claude/settings.json
new file mode 100644
index 00000000000..0ae22050977
--- /dev/null
+++ b/.claude/settings.json
@@ -0,0 +1,7 @@
+{
+ "$schema": "https://json.schemastore.org/claude-code-settings.json";,
+ "attribution": {
+ "commit": "",
+ "pr": ""
+ }
+}
diff --git a/AGENTS.md b/AGENTS.md
index 306a5d398db..8cd931ec0c8 100644
--- a/AGENTS.md
+++ b/AGENTS.md
@@ -114,6 +114,11 @@ repo. It is intentionally short and focused on day-to-day
work.
- Prefer imports over fully qualified class names (e.g., use `import
com.foo.Bar` and refer to `Bar`, not `com.foo.Bar` inline).
- Prefer targeted unit tests; use integration tests when behavior crosses
roles.
+## Commit messages
+- Do not include `Co-authored-by` trailers that reference AI tools (e.g.,
Claude, Copilot).
+ - **Why**: These trailers propagate into squash-merge commits on GitHub,
making the project history appear AI-authored rather than human-authored.
+ - **Fix**: Omit the `Co-authored-by` line entirely when committing.
+
## Checkstyle config
- Checkstyle rules and related config files live under `config/`.
- Use the Maven wrapper (`./mvnw` on Unix-like systems or `mvnw.cmd` on
Windows) to run `spotless:apply` to format code and `checkstyle:check` to
validate style.
diff --git a/CLAUDE.md b/CLAUDE.md
index 24a5068f904..c4a290e0439 100644
--- a/CLAUDE.md
+++ b/CLAUDE.md
@@ -113,6 +113,11 @@ Apache Pinot is a real-time distributed OLAP datastore for
low-latency analytics
- Prefer targeted unit tests; use integration tests when behavior crosses
roles.
- Avoid deprecated APIs in new code. If you must reference one (e.g., for
backward-compat serialization or to test the deprecated path), justify it with
a comment.
+## Commit messages
+- Do not include `Co-authored-by` trailers that reference AI tools (e.g.,
Claude, Copilot).
+ - **Why**: These trailers propagate into squash-merge commits on GitHub,
making the project history appear AI-authored rather than human-authored.
+ - **Fix**: Omit the `Co-authored-by` line entirely when committing.
+
## Pre-commit checks
Before pushing a commit, run the following checks on the affected modules and
fix any failures:
1. `./mvnw spotless:apply -pl <module>` — auto-format code.
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
