Skip to content
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

Include github.actor_id in default commit_author #354

Merged
merged 1 commit into from
Jan 11, 2025

Conversation

parkerbxyz
Copy link
Contributor

@parkerbxyz parkerbxyz commented Jan 10, 2025

This mimics the default commit author used by GitHub and matches the format used for the default commit_user_email.

Motivation: When I use the default settings, I end up with two different email addresses for the same commit author. By using the same default as GitHub, the author information remains consistent.

This mimics the default commit author used by GitHub and matches the format used for the default `commit_user_email`.
Copy link
Owner

@stefanzweifel stefanzweifel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks Parker! LGTM.
Will release this as a minor release in a few moments.

@stefanzweifel stefanzweifel merged commit e348103 into stefanzweifel:master Jan 11, 2025
2 checks passed
@parkerbxyz parkerbxyz deleted the patch-1 branch January 14, 2025 22:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants