Skip to content

graft batch 32: 1 land from backlog-1 - #313

Closed
thedancingdeveloper wants to merge 1 commit into
mainfrom
graft/batch-32
Closed

thedancingdeveloper wants to merge 1 commit into
mainfrom
graft/batch-32

Conversation

@thedancingdeveloper

Copy link
Copy Markdown
Contributor

Batch 32 of graft line backlog-1, one commit per land, oldest first, by graft-mirror (decision D26). Merge it with rebase-merge to keep the commits (D28). graft never merges or queues this PR.

  • Base: main at 214ccad (graft c262b8fb4a43dd1599d95764b2b1265d4de097d166d9bf0c1f926dfe7565d9869)
  • Lands: 1 (c80ef613e33acffaa70df3d6589bfbaa0d88ebb14f8b6683e3103ac2796730df6..c44efdac51a708f1474760dfbfb5a5ee002b901b94b5071a94d95878a474bdb15)
Changeset Author Session Work item Attestations Subject
c44efdac51a70 thedancingdeveloper ses_01M6ETYFFT5Q77S9N5QXZ4AQQK WI-3053 none graft-mirror: Claimed-Author keeps a claimed email only on allowed domains (WI-3053)

Graft-Batch: 32
Graft-Line: backlog-1
Graft-Changeset: c44efdac51a708f1474760dfbfb5a5ee002b901b94b5071a94d95878a474bdb15

…mains (WI-3053)

Drop the trailer when the claimed email matches [export.authors] or
default_author; otherwise keep the email only when its domain is on
[export] claimed_author_domains (default users.noreply.github.com), else
write the name alone. New batches only; merged commits are unchanged.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

Graft-Changeset: c44efdac51a708f1474760dfbfb5a5ee002b901b94b5071a94d95878a474bdb15
Graft-Session: ses_01M6ETYFFT5Q77S9N5QXZ4AQQK
Graft-Work-Item: WI-3053
Claimed-Author: thedancingdeveloper <306930456+thedancingdeveloper@users.noreply.github.com>
Graft-Batch: 32
@thedancingdeveloper

Copy link
Copy Markdown
Contributor Author

Closing for a fix: the review asks for tests on the suffix tricks, whitespace, missing '@' and bad config types, and for emails with more than one '@' to be refused. The fix lands on backlog-1 and rides the next batch.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant