Skip to content

Allow contributor resolution to identify github users from their noreply addresses - #345

Draft
MoralCode wants to merge 12 commits into
mainfrom
feat/resolution_shortcut_ghnoreply
Draft

MoralCode wants to merge 12 commits into
mainfrom
feat/resolution_shortcut_ghnoreply

Conversation

@MoralCode

Copy link
Copy Markdown
Contributor

Description
This PR shows what an implementation might look like if we wanted to account for github noreply user emails in the contributor resolution process since this specific email format makes it easier to connect a user to their commits with fewer API calls.

It also includes unit tested helpers for extracting and validating email addresses.

This PR fixes #256

Notes for Reviewers
Not yet tested

Signed commits

  • Yes, I signed my commits.

@MoralCode MoralCode added the waiting This change is waiting for some other changes to land first label May 26, 2026
@MoralCode MoralCode self-assigned this May 27, 2026
@MoralCode MoralCode moved this from Todo to Paused/Background in CollectOSS + 8Knot WG Feature Roadmap May 27, 2026
@MoralCode MoralCode added the needs-testing PRs that need to be tested further before they can merge label Oct 8, 2026
Signed-off-by: Adrian Edwards <adredwar@redhat.com>
Signed-off-by: Adrian Edwards <adredwar@redhat.com>
…are found

Signed-off-by: Adrian Edwards <adredwar@redhat.com>
Signed-off-by: Adrian Edwards <adredwar@redhat.com>
Signed-off-by: Adrian Edwards <adredwar@redhat.com>
…er side

We are assuming we are working with a plain email

Signed-off-by: Adrian Edwards <adredwar@redhat.com>
This helps handle the edge case where there are other words besides an email, separated by a space, in the string.

Signed-off-by: Adrian Edwards <adredwar@redhat.com>
…ext either side of the @

Signed-off-by: Adrian Edwards <adredwar@redhat.com>
Signed-off-by: Adrian Edwards <adredwar@redhat.com>
Signed-off-by: Adrian Edwards <adredwar@redhat.com>
Signed-off-by: Adrian Edwards <adredwar@redhat.com>
Signed-off-by: Adrian Edwards <adredwar@redhat.com>
@MoralCode
MoralCode force-pushed the feat/resolution_shortcut_ghnoreply branch from 5713b7a to efb6b1f Compare October 8, 2026 01:27
@MoralCode MoralCode removed the needs-testing PRs that need to be tested further before they can merge label Oct 8, 2026
@MoralCode
MoralCode marked this pull request as draft October 8, 2026 01:28
@MoralCode

Copy link
Copy Markdown
Contributor Author

so far this is just email parsing utils.

needs to implement the actual feature

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

Labels

waiting This change is waiting for some other changes to land first

Projects

Status: Paused/Background

Development

Successfully merging this pull request may close these issues.

facade: we can resolve contributors with github noreply addresses in 1 api call

2 participants