Skip to content

fix: set git user config instead of --author flag#1

Merged
wadefletch merged 1 commit intomainfrom
wade/fix-committer-identity
Apr 11, 2026
Merged

fix: set git user config instead of --author flag#1
wadefletch merged 1 commit intomainfrom
wade/fix-committer-identity

Conversation

@wadefletch
Copy link
Copy Markdown
Contributor

Summary

  • Replace --author flag on git commit with git config user.name and git config user.email set before committing
  • The --author flag only sets the author identity, not the committer identity. On GitHub Actions runners where no default git identity is configured, this causes fatal: empty ident name because git still has no committer to use.
  • Setting user.name and user.email via git config configures both author and committer, resolving the error.

Test plan

  • Deploy to a repo and trigger a fixup run on a GitHub Actions runner to confirm commits succeed without the fatal: empty ident name error

@wadefletch wadefletch merged commit 66c86a2 into main Apr 11, 2026
1 check passed
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