feat(commit): add --write-message-to-file option#731
feat(commit): add --write-message-to-file option#731woile merged 11 commits intocommitizen-tools:masterfrom crai0:write-message-to-file
Conversation
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## master #731 +/- ##
==========================================
- Coverage 97.38% 97.35% -0.04%
==========================================
Files 42 42
Lines 2029 2041 +12
==========================================
+ Hits 1976 1987 +11
- Misses 53 54 +1
Flags with carried forward coverage won't be shown. Click here to find out more.
☔ View full report in Codecov by Sentry. |
|
Great work! 💪🏻 thanks! |
|
i noticed that it's not possible to use the |
|
The Is it not possible to run other checks and then the commit? Is there a way to refactor the code in a non-breaking change? But let's open another issue and discuss it there. |
|
this would not be a breaking change if it can be configured and isn't be enabled by default. i'll open an issue for it and describe the feature in a backward compatible manner. |
|
i had to remove the hook i was using because i added it on accident but otherwise i think this can be merged, right? |
|
never mind, i couldn't stand the way the hook worked before so i made the following changes:
|
|
That's nice! Thanks for taking the time to build this. I'll take a look at it tomorrow |
Description
This pull request resolves #689 by implementing my proposal for it.
Checklist
./scripts/formatand./scripts/testlocally to ensure this change passes linter check and testExpected behavior
See #689 (comment).
Steps to Test This Pull Request
New option:
cz commit --write-message-to-file message.txtin a Git repository with staged changes and complete interactive dialogmessage.txtshould be the generated commit messagecz commitis unchangedGit hook:
docs/tutorials/auto_prepare_commit_message.mdin a Git repository with staged changesgit commitorgit commit -m "...", the usualcz commitinteractive dialog runsAdditional context
These changes supersede #249 and #250.