Skip to content
This repository was archived by the owner on Apr 12, 2024. It is now read-only.

chore(contributing): add commitizen commit message formatting #13194

Closed
wants to merge 2 commits into from
Closed

chore(contributing): add commitizen commit message formatting #13194

wants to merge 2 commits into from

Conversation

stephensauceda
Copy link
Contributor

This adds npm run commit to package.json scripts to assist in formatting git commit messages with commitizen.

Closes #12730

@kentcdodds
Copy link

Would love to see this merged! Great work @stephensauceda 👍

@Narretz
Copy link
Contributor

Narretz commented Oct 29, 2015

Looks like a worthwhile addition. Wdyt @petebacondarwin ?

@petebacondarwin
Copy link
Contributor

Sure, as long as it is in keeping with our conventions

@kentcdodds
Copy link

It is. Never fear :-) and you may also be interested in:
http://npm.im/validate-commit-msg

On Thu, Oct 29, 2015, 4:27 PM Pete Bacon Darwin notifications@github.com
wrote:

Sure, as long as it is in keeping with our conventions


Reply to this email directly or view it on GitHub
#13194 (comment).

@petebacondarwin
Copy link
Contributor

Merging

@petebacondarwin
Copy link
Contributor

Yes, this looks cool. Although I tend to use a UI tool for rebasing and updating commits :-)

@jimthedev
Copy link

@petebacondarwin Eventually I'd love to see some GUIs hook into this to be able to offer prompts. There is an open (pie in the sky) issue for it here: commitizen/cz-cli#45

The GUIs would just need to offer some hook we could call then wrap the Inquirer.js interface for prompts. Again, this may never happen, but it would be nice to offer it as an opt-in integration.

gkalpak pushed a commit to gkalpak/angular.js that referenced this pull request Nov 23, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants