Skip to content

chore(release): map brian@dralth.com to btorresgil#26319

Merged
kshitijk4poor merged 1 commit into
NousResearch:mainfrom
kshitijk4poor:chore/author-map-btorresgil-26260-salvage
May 15, 2026
Merged

chore(release): map brian@dralth.com to btorresgil#26319
kshitijk4poor merged 1 commit into
NousResearch:mainfrom
kshitijk4poor:chore/author-map-btorresgil-26260-salvage

Conversation

@kshitijk4poor

Copy link
Copy Markdown
Collaborator

What does this PR do?

Adds brian@dralth.com → btorresgil to AUTHOR_MAP in scripts/release.py.

@btorresgil authors commits as Brian Conklin <brian@dralth.com> (git config carries a different name/email than the GitHub account). GitHub's commit-author mapping correctly attributes these commits to @btorresgil based on the public-key registration, but Hermes' release attribution audit reads the raw commit email, not the GitHub mapping. Without this entry, scripts/contributor_audit.py strict mode would fail when releasing.

Why now

Prerequisite for the langfuse trace fix salvage that cherry-picks @btorresgil's commits from #22345 onto current main. That salvage PR will reference this one as a hard dependency.

Verification

import sys; sys.path.insert(0, "scripts")
import release
assert release.AUTHOR_MAP["brian@dralth.com"] == "btorresgil"

Type of Change

  • 🧹 Chore (release housekeeping)

…2345 salvage

PR NousResearch#22345 by @btorresgil authors commits as 'Brian Conklin
<brian@dralth.com>' (git config carries a different name/email than the
GitHub account). GitHub's commit-author mapping correctly attributes these
commits to @btorresgil based on the public-key registration, but Hermes'
release attribution audit reads the raw commit email, not the GitHub
mapping. Without this AUTHOR_MAP entry, salvaging NousResearch#22345 would fail
`scripts/contributor_audit.py` strict mode at release time.

Prerequisite for the langfuse trace fix salvage that cherry-picks
@btorresgil's commits onto current main.
@kshitijk4poor kshitijk4poor merged commit f199cd9 into NousResearch:main May 15, 2026
10 of 12 checks passed
@alt-glitch alt-glitch added type/refactor Code restructuring, no behavior change P3 Low — cosmetic, nice to have labels May 15, 2026
DIZ-admin pushed a commit to DIZ-admin/hermes-agent that referenced this pull request May 16, 2026
…2345 salvage (NousResearch#26319)

PR NousResearch#22345 by @btorresgil authors commits as 'Brian Conklin
<brian@dralth.com>' (git config carries a different name/email than the
GitHub account). GitHub's commit-author mapping correctly attributes these
commits to @btorresgil based on the public-key registration, but Hermes'
release attribution audit reads the raw commit email, not the GitHub
mapping. Without this AUTHOR_MAP entry, salvaging NousResearch#22345 would fail
`scripts/contributor_audit.py` strict mode at release time.

Prerequisite for the langfuse trace fix salvage that cherry-picks
@btorresgil's commits onto current main.
gweeteve pushed a commit to gweeteve/hermes-agent that referenced this pull request Jun 2, 2026
…2345 salvage (NousResearch#26319)

PR NousResearch#22345 by @btorresgil authors commits as 'Brian Conklin
<brian@dralth.com>' (git config carries a different name/email than the
GitHub account). GitHub's commit-author mapping correctly attributes these
commits to @btorresgil based on the public-key registration, but Hermes'
release attribution audit reads the raw commit email, not the GitHub
mapping. Without this AUTHOR_MAP entry, salvaging NousResearch#22345 would fail
`scripts/contributor_audit.py` strict mode at release time.

Prerequisite for the langfuse trace fix salvage that cherry-picks
@btorresgil's commits onto current main.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

P3 Low — cosmetic, nice to have type/refactor Code restructuring, no behavior change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants