-
Notifications
You must be signed in to change notification settings - Fork 48
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
Question: configuration for custom settings commit-analyzer settings #165
Comments
You use the preset If you format your commits with the eslint format won't find any commit to add to the release note because it would look for commits formatted with |
Thanks @pvdlg , yes just realized it and changed to below:
Noticed following from the logs:
|
The |
Thanks @pvdlg
|
@pvdlg thank you once more for pointing in the right direction. Removed "types" and
|
:) that did not work, |
I think release-notes-generator/index.js Line 43 in aa7fd39
release-notes-generator/index.js Line 80 in aa7fd39
So the code doing the matching could inside here?: release-notes-generator/index.js Line 5 in aa7fd39
Hopefully this is the right direction... digging in |
Okay, reading thru https://github.com/conventional-changelog/conventional-changelog/blob/master/packages/conventional-commits-parser/index.js
|
Only thing am struggling with is to include the commits without any
|
The |
@pvdlg wonder how you figured that |
In further debugging this scenario changed
But the
|
Configuring
Just use only |
okay, that's good to know. will revert to using
|
@pheenomenon have you fixed your issue? |
Hi, I recently added the below configuration for
semantic-release/commit-analyzer
. More details here: semantic-release/commit-analyzer#162After adding these the
release-notes-generator
is not creating release notes. I am looking at the config forrelease-notes-generator
and changed.releaserc
like below, but wanted to ask if I am in the right direction and if the above customcommit-analyzer
will work withrelease-notes-generator
? Currently debugging but any pointers will help.The text was updated successfully, but these errors were encountered: