feat!: remove improvement

[AndersDJohnson]

I'm proposing we consider removing the improvement type, which is no longer recommended in the Conventional Commits spec. It was in their 1.0.0-beta.4 release, but removed in 1.0.0.

It was added here in #13, as did other tools like conventional-changelog/commitlint#832. But this addition was questioned in #15, which I'd agreed with per my comments #13 (review) and #15 (comment).

This also caused some other turbulence with other tools, such as commitizen/cz-cli#681 and conventional-changelog/commitlint#832.

Maybe it's too late? 🤷‍♂

[NathanUrwin]

@AndersDJohnson I do not think it is too late to remove improvement. Going forward in the future I believe exceptions can be made for the edge case commits that include the improvement type. I agree with your comment on #15 that this project follows the conventional commits specification, and does not make any type decisions itself.

[skeggse]

The good news is that it may not have spread too far: tools like semantic-release didn't start supporting the improvement commit type (ref semantic-release/commit-analyzer#149)

[jamime]

I also came across this issue today when using commitizen with cz-conventional-changelog then using commitlint with @commitlint/config-conventional.

[skeggse]

@AndersDJohnson are there any blockers to moving forward with this change?

[AndersDJohnson]

@skeggse None other than making sure we have consensus and an admin taking the time to merge it! Almost ready?

[sc250024]

Yes! This is interfering with commits as improvement is not in the spec.

[sc250024]

I also came across this issue today when using commitizen with cz-conventional-changelog then using commitlint with @commitlint/config-conventional.

Ditto

[AndersDJohnson]

Released and published as v3.0.0!