Skip to content

chore: add Bartok9 to AUTHOR_MAP in scripts/release.py - #14447

Closed
Bartok9 wants to merge 1 commit into
NousResearch:mainfrom
Bartok9:fix/author-map-bartok9
Closed

chore: add Bartok9 to AUTHOR_MAP in scripts/release.py#14447
Bartok9 wants to merge 1 commit into
NousResearch:mainfrom
Bartok9:fix/author-map-bartok9

Conversation

@Bartok9

@Bartok9 Bartok9 commented Apr 23, 2026

Copy link
Copy Markdown
Contributor

Summary

Adds the Bartok9 contributor account to AUTHOR_MAP in scripts/release.py.

Without this mapping, the Contributor Attribution Check workflow fails with:

New contributor email(s) not in AUTHOR_MAP:
  danielrpike9@gmail.com (Bartok9)

...on every PR from this account, blocking CI green status unnecessarily.

Changes

# Bartok9 (Bartok AI agent — danielrpike9@gmail.com)
"danielrpike9@gmail.com": "Bartok9",
"259807879+Bartok9@users.noreply.github.com": "Bartok9",

Both the standard Gmail address and the GitHub noreply address are included (same pattern as other multi-email contributors in the map).

Adds both email formats for the Bartok9 contributor account:
  - danielrpike9@gmail.com → Bartok9
  - 259807879+Bartok9@users.noreply.github.com → Bartok9

Without this mapping, the Contributor Attribution Check workflow fails
with 'New contributor email(s) not in AUTHOR_MAP' on every PR from this
account, blocking CI green status unnecessarily.
@alt-glitch alt-glitch added type/refactor Code restructuring, no behavior change P3 Low — cosmetic, nice to have labels Apr 23, 2026
@teknium1

Copy link
Copy Markdown
Contributor

Thanks for the contribution @Bartok9! This is an automated hermes-sweeper review.

Both AUTHOR_MAP entries this PR adds are already present on main:

  • scripts/release.py line 200: "danielrpike9@gmail.com": "Bartok9"
  • scripts/release.py line 100: "259807879+Bartok9@users.noreply.github.com": "Bartok9"

This means the CI attribution check should already pass for your account. Closing as implemented on 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.

3 participants