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(cli): override package version #7

Merged
merged 1 commit into from
Oct 28, 2022
Merged

fix(cli): override package version #7

merged 1 commit into from
Oct 28, 2022

Conversation

cdcabrera
Copy link
Owner

What's included

  • fix(cli): override package version

Notes

  • corrects the override option by also updating package.json

How to test

Coverage and basic unit test check

  1. update the NPM packages with $ yarn
  2. $ yarn test
  3. confirm tests come back clean

Interactive unit test check

  1. update the NPM packages with $ yarn
  2. $ yarn release --dry-run -o v15.0.0
  3. confirm the console output aligns to a proper override display
Dry run CHANGELOG.md output...
Version: v15.0.0
Semver bump: override
Semver weight: XXX

## [15.0.0](https://github.com/XXXX/changelog-light/compare/XXXX...XXXX) (2022-10-28)

...

Example

...

Updates issue/story

ongoing

@cdcabrera cdcabrera added the bug Something isn't working label Oct 28, 2022
* index,  override version to npm version
* parse, display bump as override
@codecov-commenter
Copy link

Codecov Report

Merging #7 (6f917fa) into main (2daa637) will increase coverage by 0.11%.
The diff coverage is 100.00%.

❗ Current head 6f917fa differs from pull request most recent head 855d1b7. Consider uploading reports for the commit 855d1b7 to get more accurate results

Impacted file tree graph

@@            Coverage Diff             @@
##             main       #7      +/-   ##
==========================================
+ Coverage   91.11%   91.22%   +0.11%     
==========================================
  Files           5        5              
  Lines         225      228       +3     
  Branches       84       88       +4     
==========================================
+ Hits          205      208       +3     
  Misses         19       19              
  Partials        1        1              
Impacted Files Coverage Δ
src/index.js 82.60% <100.00%> (ø)
src/parse.js 96.59% <100.00%> (+0.12%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2daa637...855d1b7. Read the comment docs.

@cdcabrera cdcabrera merged commit d617632 into main Oct 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants