Clarify commit message from pr-auto-commit by moreal · Pull Request #6706 · RustPython/RustPython

coderabbitai

37-40: LGTM! Explicit git configuration improves clarity.

Adding this dedicated step ensures consistent committer identity for all auto-commits and makes the workflow more maintainable.

46-49: LGTM! Conditional commit achieves the PR objective.

The workflow now only creates a "cargo fmt" commit when formatting actually makes changes, addressing the core issue described in the PR.

57-63: LGTM! Separate commit for ruff format improves clarity.

This separate step with its own conditional commit allows distinguishing between formatting changes and import sorting changes in the commit history.

65-71: LGTM! Import sorting as a separate step provides clarity.

The separate commit for import sorting makes it clear what type of auto-formatting was applied, and explains the changes in scripts/fix_test.py.

73-79: LGTM! Separate commit for code generation is appropriate.

The distinct "Auto-generate" prefix in the commit message clearly differentiates generated code changes from formatting changes.

84-84: LGTM! Improved change detection logic.

Comparing HEAD to the original PR head SHA correctly detects whether any of the conditional formatting/generation steps created commits, which is more robust than checking the working directory state.

90-93: LGTM! Step name accurately reflects its purpose.

The renamed step correctly indicates it only pushes commits made by previous steps, not creating new commits itself.