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

add useVersion support to npm changelog #2347

Merged
merged 1 commit into from
Apr 3, 2023

Conversation

ds300
Copy link
Contributor

@ds300 ds300 commented Mar 23, 2023

What Changed

I added the useVersion override parameter to the changelog lifecycle events so that changelogs can be compiled with the same version number as the packages themselves. I also made the npm plugin respect this option.

Why

At the moment, if I pass useVersion it is used successfully when doing versioning and publishing but not when compiling the changelog, which ignores the override and uses the regular bumped version.

Todo:

  • Add tests
  • Add docs (useVersion is not currently documented)

Change Type

Indicate the type of change your pull request is:

  • documentation
  • patch
  • minor
  • major

const title = `v${inc(lernaPackage.version, bump as ReleaseType)}`;
const title = `v${
useVersion || inc(lernaPackage.version, bump as ReleaseType)
}`;
Copy link
Contributor Author

Choose a reason for hiding this comment

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

this is the only semantic change in this file, everything else was prettier

@hipstersmoothie hipstersmoothie added the minor Increment the minor version when merged label Apr 3, 2023
@hipstersmoothie hipstersmoothie merged commit 2e9e648 into intuit:main Apr 3, 2023
@github-actions github-actions bot removed the minor Increment the minor version when merged label Apr 3, 2023
@github-actions
Copy link

github-actions bot commented Apr 3, 2023

🚀 PR was released in v10.44.0 🚀

@github-actions github-actions bot added the released This issue/pull request has been released. label Apr 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
released This issue/pull request has been released.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants