Skip to content
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

Combining both release-as and prerelease now doesn't break package version #767

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

kf6kjg
Copy link

@kf6kjg kf6kjg commented Jun 1, 2021

Previously, as shown in #542, using both options at the same time resulted in the bump file's version being set to null.

This was validated in the first of the new unit tests. The other unit tests simply check that the rest of the behavior is correct in this case.

With this combination the user can override the version, the purpose of release-as, while still starting or continuing a pre-release, the purpose of of the prerelease.

Fixes #542

Previously, as shown in conventional-changelog#542, using both options at the same time resulted in the bump file's version being set to null.

This was validated in the first of the new unit tests. The other unit tests simply check that the rest of the behavior is correct in this case.

With this combination the user can override the version, the purpose of `release-as`, while still starting or continuing a pre-release, the purpose of of the `prerelease`.

Fixes conventional-changelog#542
@TimothyJones
Copy link
Contributor

Since this package is now deprecated due to maintainer availability, I've forked it here, and released a version that includes this fix.

You can get it from npm here, by installing commit-and-tag-version with version equal to or greater than 9.6.0.

Thanks for making this easy by making a PR!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

Using both prelease and release-as parameters, the version number becomes null
2 participants