refactor: trim trailing whitespace from commit message#145
Conversation
Ensures cleaner commit messages by stripping trailing spaces, tabs, and carriage returns from each line before processing. Added a helper function with corresponding unit tests to verify behavior. This was causing some glitches with bubbletea text component too.
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
There was a problem hiding this comment.
Code Review
This pull request introduces a helper function trimTrailingSpaces to remove trailing spaces, tabs, and carriage returns from each line of a string, and applies it to the commitMessage in the Update method of the UI model. Additionally, comprehensive unit tests have been added to verify the behavior of this helper function. There are no review comments, and I have no feedback to provide.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
Coverage Report for CI Build 28683367083Coverage increased (+0.4%) to 31.551%Details
Uncovered ChangesNo uncovered changes found. Coverage RegressionsNo coverage regressions found. Coverage Stats
💛 - Coveralls |
Ensures cleaner commit messages by stripping trailing spaces, tabs, and carriage returns from each line before processing. Added a helper function with corresponding unit tests to verify behavior.
This was causing some glitches with bubbletea text component too.