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

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

Merged
merged 9 commits into from
Aug 26, 2024

Conversation

mrgrain
Copy link
Contributor

@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
@iliapolo iliapolo added the pr/do-not-merge ⚙️ used by automation label Aug 26, 2024
Copy link
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.

src/version.ts Outdated Show resolved Hide resolved
Co-authored-by: Eli Polonsky <[email protected]>
src/version.ts Outdated Show resolved Hide resolved
@mrgrain mrgrain removed the pr/do-not-merge ⚙️ used by automation label Aug 26, 2024
@mergify mergify bot merged commit 4c78131 into main Aug 26, 2024
16 checks passed
@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