You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@thrift.apache.org by "Jens-G (via GitHub)" <gi...@apache.org> on 2026/08/28 22:52:40 UTC

[PR] Forbid tool-internal links in commit messages and PR text [thrift]

Jens-G opened a new pull request, #3758:
URL: https://github.com/apache/thrift/pull/3758

   Some AI assistants append a session or conversation URL to every commit message and PR body by default — `Claude-Session:`, `https://claude.ai/code/session_…`, `https://chatgpt.com/c/…` and equivalents.
   
   Such a link does not belong in this project's history:
   
   - it is **permanent** once it reaches git, and a force-push does not undo it — the superseded commit stays reachable by SHA;
   - it means nothing to a reviewer, who cannot open it;
   - it can point at material that is **not public**, including work still under embargo per §6.
   
   §4 already says what attribution belongs in a trailer block. This says what does not, and §8's pre-PR checklist gains the matching line. The convention in this repository's history is already the attribution line alone — 45+ AI-assisted commits on master carry `Co-authored-by:` and none carry a session link — so this documents existing practice rather than changing it.
   
   Docs only; no code is touched. Prior AGENTS.md changes have gone in without a JIRA ticket (`59fc4113e`, `bcd91a392`, `a1d117afb`), so no ticket here either — and those also used `[skip ci]`, which would be reasonable on merge for a markdown-only change.
   
   🤖 Generated with [Claude Code](https://claude.com/claude-code)


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@thrift.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] Forbid tool-internal links in commit messages and PR text [thrift]

Posted by "Jens-G (via GitHub)" <gi...@apache.org>.
Jens-G merged PR #3758:
URL: https://github.com/apache/thrift/pull/3758


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@thrift.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org