-
Notifications
You must be signed in to change notification settings - Fork 11
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
feat: Extract conventional commit messages for the changelog #314
Conversation
|
||
Reviewed-by: Z | ||
Refs: #123 | ||
## feat(lang) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we might want empty lines before and after h2?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, those would be great.
I need to look at the failures |
@krlmlr now ready for review |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks. Overall, I think it would have been sufficient to do something like
- docs: correct spelling
- chore!: drop support for Node 6
- BREAKING CHANGE: use JavaScript features not available in Node 6.
- ...
for the highlighted part. Baby steps and such. We can flesh out details later.
|
||
Reviewed-by: Z | ||
Refs: #123 | ||
## feat(lang) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, those would be great.
## docs | ||
correct spelling of CHANGELOG | ||
|
||
## chore! | ||
drop support for Node 6 | ||
|
||
BREAKING CHANGE: use JavaScript features not available in Node 6. | ||
|
||
## feat(api)! | ||
feat(api)!: send an email to the customer when a product is shipped |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would have expected those to look a bit like the following, but let's tackle this in a future PR:
## docs | |
correct spelling of CHANGELOG | |
## chore! | |
drop support for Node 6 | |
BREAKING CHANGE: use JavaScript features not available in Node 6. | |
## feat(api)! | |
feat(api)!: send an email to the customer when a product is shipped | |
## docs | |
- correct spelling of CHANGELOG. | |
## chore | |
- drop support for Node 6. | |
## Breaking change | |
- use JavaScript features not available in Node 6. | |
## Breaking change | |
### api | |
- send an email to the customer when a product is shipped | |
## feat | |
### api | |
- send an email to the customer when a product is shipped |
Thanks! |
No description provided.