Repository navigation
docs: preserve contributor Git identity when committing - #828
Open
thehobbit85 wants to merge 1 commit into
Open
thehobbit85 wants to merge 1 commit into
thehobbit85 wants to merge 1 commit into
Conversation
thehobbit85
force-pushed
the
codex/contributor-git-identity
branch
10 times, most recently
from
November 2, 2026 17:43
3da5ed3 to
819ae88
Compare
thehobbit85
force-pushed
the
codex/contributor-git-identity
branch
3 times, most recently
from
November 5, 2026 06:04
1221739 to
fdaf199
Compare
Keep committing opt-in and retain every current repository rule, including the Front-ends notice contract. Use the contributor's author, committer and signing configuration instead of forcing the repository owner's identity.
thehobbit85
force-pushed
the
codex/contributor-git-identity
branch
from
November 13, 2026 06:18
fdaf199 to
35c55d6
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Main still forces every contributor to use a fixed Git identity. This documentation-only change preserves each contributor's configured author, committer and signing identity while retaining the requirement to commit only when asked.
Conflict refresh — 2026-11-13 UTC
Rebased
fdaf199932af501c7a44e9562f9bfc3a97cb3b03onto90b7cefd1d572ab05859c73930b436c4040cd25cas35c55d6b3b3981134f2f4b44291c89c81b640e02. The one-line conflict resolution retains main's complete currentarea/component: summarycommit-message rule, its developer-loop link and theFront-ends:requirement. The remaining main diff changes only the forced identity phrase. All earlier proposal and original commit context is preserved below.Fresh exact-head validation:
DM_GAME=010015100B514000 /opt/homebrew/bin/python3 -B -m unittest dm.tools.tests.test_docspassed all six current documentation checks. Main ancestry, full remaining diff, range-diff andgit diff --checkpassed. No runtime/build/game/shader change or game validation applies. Remote CI on this replacement head is pending verification.Historical proposal, validation and original commit context
The previous description follows in full. Its results apply to the revisions it records.
Rebased onto main
f8a5618d5f505a4c0a0bcac7c413872d3a53c009; candidatefdaf199932af501c7a44e9562f9bfc3a97cb3b03. Main still tells every contributor to commit under a fixed author/email. This PR changes only that sentence to preserve the contributor's configured author, committer and signing identity, retaining main's complete Front-ends notice requirement and every other repository rule.Clean rebase; unchanged feature patch. Fresh exact-candidate check:
DM_GAME=010015100B514000 /opt/homebrew/bin/python3 -B -m unittest -v dm.tools.tests.test_docspassed 11 tests;git diff --checkpassed. Documentation only; no runtime/game/shader/build impact. Earlier evidence and original commit context are retained as historical below.Historical PR description and original commit context (preserved)
Fixes #827. Preserve the requirement to commit only when asked while using the contributor's configured author, committer and signing identity.
Current
122173941a81c4fd1b92776fefaddf051bd65917is exactly one commit directly atop975bb4c14456d145eb3d969814534eecdd996ce9. It has the identical tree tested before the squash atf15e71f161ae3c3313cb95d303f7f19214428af7. The complete main diff is one instruction substitution; every other current rule, including theFront-ends:line, is byte-preserved. All 11 documentation tests passed;git diff --check, main ancestry, parent/count and tree-equality proofs pass. No runtime, generated artifact, agent plan or baseline changes are included.Earlier description and historical validation
Fixes #827.
Current revision
5886f856b9e15636a8c482c7380157dd2fae95d5, rebased onto upstream mainbd98317eecfe832b40d401219c15207a398e78dd. Range-diff and the remaining diff were reviewed; main is an ancestor.Fresh focused validation on this revision: 11 tests passed. The remaining diff changes only the contributor-identity instruction. It retains the requirement to commit only when asked.
Remote CI for this replacement head is pending verification by the CI agent; earlier successful checks do not validate it. Current checks.
Prior revision context and validation
The following evidence predates this rebase and retains its stated revisions and limitations.
AGENTS.mdforces commits to use the repository owner's identity. Preserve the requirement to commit only when asked while using each contributor's configured author, committer and signing identity. Main still has the forced-identity rule; #691 changes a separate instruction.Revision
819ae88308ac38ae5c2d245f622c5e6d5969f817includes main65501fbbaf8526f7aab0a1edd028135888590f8e. Main still instructs contributors to use the repository owner’s Git identity.Validation on
819ae88308ac.python3 -m unittest dm.tools.tests.test_docs— 11 tests passed.GitHub checks for this revision
Original commit context (historical, preserved through squash)
These messages preserve the original rationale, attribution, validation limits and interface notes. Any references to earlier source or test results remain historical; the current validation section and tree proof above identify the integrated revision.
Original published PR commits:
Rebased/corrective commits before this squash: