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: print version number that updater.writeVersion returns #11

Merged
merged 1 commit into from
May 25, 2022

Conversation

TimothyJones
Copy link
Member

Copy of conventional-changelog#670

BEGIN_COMMIT_OVERRIDE
fix: When a custom updater reports a version other than the new computed semver, that version is now correctly reported in log output
END_COMMIT_OVERRIDE


For custom updaters that derive the new version from the file contents (e.g. newVersion = oldVersion + 1), the log output was showing the new computed semver version, rather than the actual version applied by the updater. This PR corrects the output for such custom updaters.

Closes expo-community/standard-version-expo#18.
Closes expo-community/standard-version-expo#10.

@codecov-commenter
Copy link

Codecov Report

❗ No coverage uploaded for pull request base (master@f6a7430). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #11   +/-   ##
=========================================
  Coverage          ?   97.62%           
=========================================
  Files             ?       26           
  Lines             ?     1177           
  Branches          ?        0           
=========================================
  Hits              ?     1149           
  Misses            ?       28           
  Partials          ?        0           

📣 Codecov can now indicate which changes are the most critical in Pull Requests. Learn more

@TimothyJones TimothyJones merged commit f2e83bf into absolute-version:master May 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants