BobAnkh / auto-generate-changelog

A Github Action to generate CHANGELOG automatically according to conventional commits. Feel free to contribute!
Apache License 2.0
81 stars 22 forks source link

[enhancement] add an option to choose whether to use `<scope>` #11

Closed BobAnkh closed 2 years ago

BobAnkh commented 4 years ago

Is your feature request related to a problem? Please describe

The changelog generater is designed for commits that follows angularjs's convention. The <header> should be like this: <type>(<scope>): <subject>. However, <scope> is optional in many projects, so I think it is better for users to choose if generater detects <scope>

Description of the new feature/enhancement

An new option to choose if detect <scope> in commit message <header>. It is set to true be default, which means to detect it.

Proposed technical implementation details (optional)

If this new option is set to false, then the program will match <type>: or <type>(<scope>): but ignore <scoope>.

Additional context

issue-label-bot[bot] commented 4 years ago

Issue-Label Bot is automatically applying the label enhancement to this issue, with a confidence of 0.98. Please mark this comment with :thumbsup: or :thumbsdown: to give our bot feedback!

Links: app homepage, dashboard and code for this bot.

BobAnkh commented 3 years ago

I think we should list commits without <scope> in a part called others when this new option is set to true, as <scope> is often an optional part to developers.

gitmagic-app[bot] commented 3 years ago

Everything looks good :thumbsup:

sruehl commented 2 years ago

this one defines it as optional https://www.conventionalcommits.org/en/v1.0.0/

sruehl commented 2 years ago

related bug https://github.com/BobAnkh/auto-generate-changelog/issues/73

sruehl commented 2 years ago

so reading through https://github.com/angular/angular/blob/22b96b9/CONTRIBUTING.md#-commit-message-guidelines it is stated that The header is mandatory and the scope of the header is optional. So even in the original angular spec it's defined as optional. So I would suggest closing this(https://github.com/BobAnkh/auto-generate-changelog/issues/11) issue in favor of https://github.com/BobAnkh/auto-generate-changelog/issues/73 which fixes this "bug" with https://github.com/BobAnkh/auto-generate-changelog/pull/75

BobAnkh commented 2 years ago

@sruehl

When I first created this repo, I suppose myself to commit all with a scope. But I think it is also very important to pick up commits without scope, so I open this issue.

You propose a good suggestion. I will take into account of this when I am free this week. Thank you very much!

github-actions[bot] commented 2 years ago

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue.