-
Notifications
You must be signed in to change notification settings - Fork 25
feat: add support for Co-authored-by commit trailers with username correlation #373
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Changes from 4 commits
e5ddef5
0a5bba8
f206280
9798816
4454fe7
211543e
1fabe84
b14feb0
306e4fc
75b1ede
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change | ||||||||||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| @@ -1,6 +1,7 @@ | ||||||||||||||||||||||||
| # pylint: disable=broad-exception-caught | ||||||||||||||||||||||||
| """This file contains the main() and other functions needed to get contributor information from the organization or repository""" | ||||||||||||||||||||||||
|
|
||||||||||||||||||||||||
| import re | ||||||||||||||||||||||||
| from typing import List | ||||||||||||||||||||||||
|
|
||||||||||||||||||||||||
| import auth | ||||||||||||||||||||||||
|
|
@@ -27,6 +28,7 @@ def main(): | |||||||||||||||||||||||
| end_date, | ||||||||||||||||||||||||
| sponsor_info, | ||||||||||||||||||||||||
| link_to_profile, | ||||||||||||||||||||||||
| acknowledge_coauthors, | ||||||||||||||||||||||||
| ) = env.get_env_vars() | ||||||||||||||||||||||||
|
|
||||||||||||||||||||||||
| # Auth to GitHub.com | ||||||||||||||||||||||||
|
|
@@ -46,7 +48,13 @@ def main(): | |||||||||||||||||||||||
|
|
||||||||||||||||||||||||
| # Get the contributors | ||||||||||||||||||||||||
| contributors = get_all_contributors( | ||||||||||||||||||||||||
| organization, repository_list, start_date, end_date, github_connection, ghe | ||||||||||||||||||||||||
| organization, | ||||||||||||||||||||||||
| repository_list, | ||||||||||||||||||||||||
| start_date, | ||||||||||||||||||||||||
| end_date, | ||||||||||||||||||||||||
| github_connection, | ||||||||||||||||||||||||
| ghe, | ||||||||||||||||||||||||
| acknowledge_coauthors, | ||||||||||||||||||||||||
| ) | ||||||||||||||||||||||||
|
|
||||||||||||||||||||||||
| # Check for new contributor if user provided start_date and end_date | ||||||||||||||||||||||||
|
|
@@ -60,6 +68,7 @@ def main(): | |||||||||||||||||||||||
| end_date=start_date, | ||||||||||||||||||||||||
| github_connection=github_connection, | ||||||||||||||||||||||||
| ghe=ghe, | ||||||||||||||||||||||||
| acknowledge_coauthors=acknowledge_coauthors, | ||||||||||||||||||||||||
| ) | ||||||||||||||||||||||||
| for contributor in contributors: | ||||||||||||||||||||||||
| contributor.new_contributor = contributor_stats.is_new_contributor( | ||||||||||||||||||||||||
|
|
@@ -103,6 +112,7 @@ def get_all_contributors( | |||||||||||||||||||||||
| end_date: str, | ||||||||||||||||||||||||
| github_connection: object, | ||||||||||||||||||||||||
| ghe: str, | ||||||||||||||||||||||||
| acknowledge_coauthors: bool = False, | ||||||||||||||||||||||||
| ): | ||||||||||||||||||||||||
| """ | ||||||||||||||||||||||||
| Get all contributors from the organization or repository | ||||||||||||||||||||||||
|
|
@@ -113,6 +123,8 @@ def get_all_contributors( | |||||||||||||||||||||||
| start_date (str): The start date of the date range for the contributor list. | ||||||||||||||||||||||||
| end_date (str): The end date of the date range for the contributor list. | ||||||||||||||||||||||||
| github_connection (object): The authenticated GitHub connection object from PyGithub | ||||||||||||||||||||||||
| ghe (str): The GitHub Enterprise URL to use for authentication | ||||||||||||||||||||||||
| acknowledge_coauthors (bool): Whether to acknowledge co-authors from commit messages | ||||||||||||||||||||||||
|
|
||||||||||||||||||||||||
| Returns: | ||||||||||||||||||||||||
| all_contributors (list): A list of ContributorStats objects | ||||||||||||||||||||||||
|
|
@@ -130,7 +142,9 @@ def get_all_contributors( | |||||||||||||||||||||||
| all_contributors = [] | ||||||||||||||||||||||||
| if repos: | ||||||||||||||||||||||||
| for repo in repos: | ||||||||||||||||||||||||
| repo_contributors = get_contributors(repo, start_date, end_date, ghe) | ||||||||||||||||||||||||
| repo_contributors = get_contributors( | ||||||||||||||||||||||||
| repo, start_date, end_date, ghe, acknowledge_coauthors | ||||||||||||||||||||||||
| ) | ||||||||||||||||||||||||
| if repo_contributors: | ||||||||||||||||||||||||
| all_contributors.append(repo_contributors) | ||||||||||||||||||||||||
|
|
||||||||||||||||||||||||
|
|
@@ -140,20 +154,61 @@ def get_all_contributors( | |||||||||||||||||||||||
| return all_contributors | ||||||||||||||||||||||||
|
|
||||||||||||||||||||||||
|
|
||||||||||||||||||||||||
| def get_contributors(repo: object, start_date: str, end_date: str, ghe: str): | ||||||||||||||||||||||||
| def get_coauthors_from_message(commit_message: str) -> List[str]: | ||||||||||||||||||||||||
| """ | ||||||||||||||||||||||||
| Extract co-author usernames from a commit message. | ||||||||||||||||||||||||
|
|
||||||||||||||||||||||||
| Co-authored-by trailers follow the format: | ||||||||||||||||||||||||
| Co-authored-by: Name <email> | ||||||||||||||||||||||||
| Or with a GitHub username: | ||||||||||||||||||||||||
| Co-authored-by: Name <username@users.noreply.github.com> | ||||||||||||||||||||||||
|
|
||||||||||||||||||||||||
| Args: | ||||||||||||||||||||||||
| commit_message (str): The commit message to parse | ||||||||||||||||||||||||
|
|
||||||||||||||||||||||||
| Returns: | ||||||||||||||||||||||||
| List[str]: List of GitHub usernames extracted from co-author trailers | ||||||||||||||||||||||||
| """ | ||||||||||||||||||||||||
| # Match Co-authored-by trailers - case insensitive | ||||||||||||||||||||||||
| # Format: Co-authored-by: Name <email> | ||||||||||||||||||||||||
| pattern = r"Co-authored-by:\s*[^<]*<([^>]+)>" | ||||||||||||||||||||||||
| matches = re.findall(pattern, commit_message, re.IGNORECASE) | ||||||||||||||||||||||||
|
|
||||||||||||||||||||||||
| usernames = [] | ||||||||||||||||||||||||
| for email in matches: | ||||||||||||||||||||||||
| # Check if it's a GitHub noreply email format: username@users.noreply.github.com | ||||||||||||||||||||||||
| noreply_pattern = r"^(\d+\+)?([^@]+)@users\.noreply\.github\.com$" | ||||||||||||||||||||||||
| noreply_match = re.match(noreply_pattern, email) | ||||||||||||||||||||||||
| if noreply_match: | ||||||||||||||||||||||||
| usernames.append(noreply_match.group(2)) | ||||||||||||||||||||||||
| return usernames | ||||||||||||||||||||||||
|
|
||||||||||||||||||||||||
|
|
||||||||||||||||||||||||
| def get_contributors( | ||||||||||||||||||||||||
| repo: object, | ||||||||||||||||||||||||
| start_date: str, | ||||||||||||||||||||||||
| end_date: str, | ||||||||||||||||||||||||
| ghe: str, | ||||||||||||||||||||||||
| acknowledge_coauthors: bool = False, | ||||||||||||||||||||||||
| ): | ||||||||||||||||||||||||
| """ | ||||||||||||||||||||||||
| Get contributors from a single repository and filter by start end dates if present. | ||||||||||||||||||||||||
|
|
||||||||||||||||||||||||
| Args: | ||||||||||||||||||||||||
| repo (object): The repository object from PyGithub | ||||||||||||||||||||||||
| start_date (str): The start date of the date range for the contributor list. | ||||||||||||||||||||||||
| end_date (str): The end date of the date range for the contributor list. | ||||||||||||||||||||||||
| ghe (str): The GitHub Enterprise URL to use for authentication | ||||||||||||||||||||||||
| acknowledge_coauthors (bool): Whether to acknowledge co-authors from commit messages | ||||||||||||||||||||||||
|
|
||||||||||||||||||||||||
| Returns: | ||||||||||||||||||||||||
| contributors (list): A list of ContributorStats objects | ||||||||||||||||||||||||
| """ | ||||||||||||||||||||||||
| all_repo_contributors = repo.contributors() | ||||||||||||||||||||||||
| contributors = [] | ||||||||||||||||||||||||
| # Track usernames already added as contributors | ||||||||||||||||||||||||
| contributor_usernames = set() | ||||||||||||||||||||||||
|
||||||||||||||||||||||||
|
|
||||||||||||||||||||||||
| try: | ||||||||||||||||||||||||
| for user in all_repo_contributors: | ||||||||||||||||||||||||
| # Ignore contributors with [bot] in their name | ||||||||||||||||||||||||
|
|
@@ -187,6 +242,15 @@ def get_contributors(repo: object, start_date: str, end_date: str, ghe: str): | |||||||||||||||||||||||
| "", | ||||||||||||||||||||||||
| ) | ||||||||||||||||||||||||
| contributors.append(contributor) | ||||||||||||||||||||||||
| contributor_usernames.add(user.login) | ||||||||||||||||||||||||
|
|
||||||||||||||||||||||||
| # Get co-authors from commit messages if enabled | ||||||||||||||||||||||||
| if acknowledge_coauthors: | ||||||||||||||||||||||||
| coauthor_contributors = get_coauthor_contributors( | ||||||||||||||||||||||||
| repo, start_date, end_date, ghe, contributor_usernames | ||||||||||||||||||||||||
| ) | ||||||||||||||||||||||||
| contributors.extend(coauthor_contributors) | ||||||||||||||||||||||||
|
||||||||||||||||||||||||
| contributors.extend(coauthor_contributors) | |
| # Avoid adding duplicate contributors for the same username within this repository | |
| filtered_coauthors = [] | |
| for coauthor in coauthor_contributors: | |
| username = getattr(coauthor, "username", None) or getattr( | |
| coauthor, "login", None | |
| ) | |
| if username and username not in contributor_usernames: | |
| filtered_coauthors.append(coauthor) | |
| contributor_usernames.add(username) | |
| contributors.extend(filtered_coauthors) |
Copilot
AI
Dec 31, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Bot accounts listed as co-authors via Co-authored-by: trailers will be included in the contributor list, while bot accounts that are regular contributors are filtered out (line 250). Consider applying the same bot filtering logic to co-authors for consistency by checking if "[bot]" is in the username before adding them to coauthor_counts.
| for username in coauthors: | |
| for username in coauthors: | |
| # Skip bot accounts for consistency with regular contributor filtering | |
| if "[bot]" in username.lower(): | |
| continue |
Uh oh!
There was an error while loading. Please reload this page.