Skip to content

fix(release): use commit-and-tag-version instead of deprecated standard-version#3801

Merged
mergify[bot] merged 9 commits into
mainfrom
mrgrain/fix/commit-and-tag-version
Aug 26, 2024
Merged

fix(release): use commit-and-tag-version instead of deprecated standard-version#3801
mergify[bot] merged 9 commits into
mainfrom
mrgrain/fix/commit-and-tag-version

Conversation

@mrgrain
Copy link
Copy Markdown
Collaborator

@mrgrain mrgrain commented Aug 24, 2024

Replace the deprecated standard-version with the drop-in fork commit-and-tag-version.

commit-and-tag-version is a fork of standard-version. Because of maintainer availability, standard-version was deprecated on 15th May 2022. The previous maintainer recommends release-please as an alternative for those who are using GitHub actions. This fork exists for those who can't switch to release-please, or who would like to continue using standard-version.

Fixes a lot of outdated dependencies.


By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@mrgrain mrgrain force-pushed the mrgrain/fix/commit-and-tag-version branch from 0a27076 to 288635d Compare August 24, 2024 20:12
mrgrain and others added 2 commits August 24, 2024 21:32
Signed-off-by: github-actions <github-actions@github.com>
Copy link
Copy Markdown
Contributor

@iliapolo iliapolo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Minor. Merge at will.

Comment thread src/version.ts Outdated
Co-authored-by: Eli Polonsky <epolon@amazon.com>
Comment thread src/version.ts Outdated
mrgrain and others added 4 commits August 26, 2024 12:56
Signed-off-by: github-actions <github-actions@github.com>
Signed-off-by: github-actions <github-actions@github.com>
@mergify mergify Bot merged commit 4c78131 into main Aug 26, 2024
@mergify mergify Bot deleted the mrgrain/fix/commit-and-tag-version branch August 26, 2024 12:43
mergify Bot pushed a commit that referenced this pull request Aug 27, 2024
Fixes #3806

The PR #3801 modified the `Version` class by moving the addition of the version bump dependency from `NodeProject` to `Version`. While this makes sense intuitively, the `Version` class was previously written in a way that could be run against any project type provided the running environment had npx installed

The fix is to only add the `bumpPackage` dependency when we have a `NodePackage` available. Also adds a test and comment to prevent a future regression.

---
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants