Skip to content

npm run commit ignores body message #1774

Closed

Description

@finetjul

Using npm run commit
I first enter no long description. But then I say it affects open issue, it requires long description.
The texted entered then never ends up in the commit body:

$npm run commit
? Select the type of change that you're committing: feat:     A new feature
? What is the scope of this change (e.g. component or file name): (press enter to skip) foo
? Write a short, imperative tense description of the change (max 89 chars):
 (7) add foo
? Provide a longer description of the change: (press enter to skip)

? Are there any breaking changes? No
? Does this change affect any open issues? Yes
? If issues are closed, the commit requires a body. Please enter a longer description of the commit itself:
 My very long description will never go into the commit body
? Add issue references (e.g. "fix #123", "re #123".):
 fix #123

Content of the commit

$ git log -1
commit e5912a16763fce65f2fc50a94abfe29ae14d87f6 (HEAD -> 123-foo)
Author: Julien Finet <julien.finet@kitware.com>
Date:   Wed Mar 10 17:28:51 2021 +0100

    feat(foo): add foo

    fix #123

Where is "My very long description will never go into the commit body"?
I'm on windows if that makes a difference.

Activity

  1. jourdain commented on Mar 10, 2021

    @jourdain
    Collaborator

    Why didn't you provide it at Provide a longer description of the change:?

  2. finetjul commented on Mar 10, 2021

    @finetjul
    MemberAuthor

    I did not think it was deserving a longer description. I guess this is not really the issue here. Or said differently, yes, there is a work-around (i.e. specify a longer description at Provide a longer description of the change:).

  3. jourdain commented on Mar 10, 2021

    @jourdain
    Collaborator

    My understanding was that commitizen was telling you that you should have provided a longer description, but it was not really asking for anything at that time (too late).

    If you think that is an issue, you should post it on the commitizen project directly.

  4. finetjul commented on Mar 10, 2021

    @finetjul
    MemberAuthor
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Labels

No labels
No labels

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions