conventional-changelog / commitlint

📓 Lint commit messages
https://commitlint.js.org
MIT License
16.63k stars 892 forks source link

Ensure a smooth transition from Angular.js to Angular conventions #146

Open marionebl opened 6 years ago

marionebl commented 6 years ago

Update 2017-11-30

As discussed with the conventional-changelog team in https://github.com/conventional-changelog/standard-version/pull/220, the way forward for this will be a soft fork from the Angular convention, where we aim for

For this reason I created the config-conventional package with suggestions what the fork should look like. Work related to conventional-changelog is tracked in https://github.com/conventional-changelog/standard-version/pull/220.


The conventions underpinning config-angular changed with the transition from Angular.js to Angular.

commitlint reflects this in config-angular@5.

This has not been reflected in the wider ecosystem so far, leading to incompatibilites. To reinstantiate compat I set the latest tag back to config-angular@4 and tagged config-angular@5 as next.

❯ npm show @commitlint/config-angular .dist-tags
{ latest: '4.3.0', next: '5.0.1' }

❯ npm show commitlint-config-angular .dist-tags
{ latest: '4.3.0', next: '5.0.1' }

Open issues

Related

honzajavorek commented 6 years ago

https://greenkeeper.io/ creates chore commits as well.

honzajavorek commented 6 years ago

Also, I've got following report on the automatic commit made by Greenkeeper:

⧗   input: chore(package): update @commitlint/config-angular to version 5.0.1
✖   subject must not be sentence-case,start-case,pascal-case,upper-case [subject-case]
✖   type must be one of [build, ci, docs, feat, fix, perf, refactor, revert, style, test] [type-enum]
✖   found 2 problems, 0 warnings

I don't understand the first one. Why is the commit message chore(package): update @commitlint/config-angular to version 5.0.1 invalid? I believe this was not an issue previously.

marionebl commented 6 years ago

@honzajavorek Thanks for reporting, I'll have a closer look at the first one - looks like a bug.

✖   subject must not be sentence-case,start-case,pascal-case,upper-case [subject-case]
marionebl commented 6 years ago

I don't understand the first one. Why is the commit message chore(package): update @commitlint/config-angular to version 5.0.1 invalid? I believe this was not an issue previously.

I tried to update the config to the newest conventions as described in the Angular project and released this prematurely. (they forbid "chore" now)

I reset the latest tag to config-angular@4, so we should have some time now to sort this out.