-
Notifications
You must be signed in to change notification settings - Fork 443
Open
Description
cz-conventional-changelog/engine.js
Lines 139 to 154 in e7bd546
| { | |
| type: 'input', | |
| name: 'breakingBody', | |
| default: '-', | |
| message: | |
| 'A BREAKING CHANGE commit requires a body. Please enter a longer description of the commit itself:\n', | |
| when: function(answers) { | |
| return answers.isBreaking && !answers.body; | |
| }, | |
| validate: function(breakingBody, answers) { | |
| return ( | |
| breakingBody.trim().length > 0 || | |
| 'Body is required for BREAKING CHANGE' | |
| ); | |
| } | |
| }, |
answers.breakingBody does not affect the commit message at all.
cz-conventional-changelog/engine.js
Line 207 in e7bd546
| var body = answers.body ? wrap(answers.body, wrapOptions) : false; |
I wrote breakingBody but found this doesn't make any sense. I read the code and I can't understand the meaning of this existence.
Shouldn't the body be combined like this?
var b = answers.body || answers.breakingBody || answers.issuesBody
var body = b ? wrap(b , wrapOptions) : false; s14k51
Metadata
Metadata
Assignees
Labels
No labels