-
Notifications
You must be signed in to change notification settings - Fork 800
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Feature/append publish message after success #47
Feature/append publish message after success #47
Conversation
Changes Unknown when pulling faed42a on tomchentw:feature/append-publish-message-after-success into * on conventional-changelog:master*. |
Looks good :) maybe a bit less verbose? |
@stevemao any suggested wording? I'm okay with any wording here. Just let me know so that I can change it. |
I'd let @bcoe to make the call. But I'd probably just say "Run |
Updated for now. |
Changes Unknown when pulling 4fdbfe3 on tomchentw:feature/append-publish-message-after-success into * on conventional-changelog:master*. |
@tomchentw love it \o/ |
Thanks! |
Mind to release a new version on npm? |
@tomchentw give this a shot:
thanks a ton for the contribution. |
@bcoe yeah it's working! Thanks :) |
Thanks for creating this package! It's not only awesome but also aligned with my best practices so I use it like at home. Just one more thing to add: