chglog icon indicating copy to clipboard operation
chglog copied to clipboard

ignore tags from other branches

Open masonkatz opened this issue 1 year ago • 5 comments

addresses issue #99

TestOffBranchTags creates a repo with a master and develop branch, and alternates commits on the branches. The resulting changelog from the master branch should only have those versions/commits.

This is only a test that shows the issue.

masonkatz avatar Jul 15 '22 22:07 masonkatz

Added potential fix to pull request as independent commit.

masonkatz avatar Jul 17 '22 22:07 masonkatz

Codecov Report

Merging #100 (45bee1b) into master (4864bcc) will increase coverage by 2.98%. The diff coverage is 65.51%.

:exclamation: Current head 45bee1b differs from pull request most recent head 5dc9271. Consider uploading reports for the commit 5dc9271 to get more accurate results

@@            Coverage Diff             @@
##           master     #100      +/-   ##
==========================================
+ Coverage   22.24%   25.22%   +2.98%     
==========================================
  Files          15       15              
  Lines         625      654      +29     
==========================================
+ Hits          139      165      +26     
- Misses        466      467       +1     
- Partials       20       22       +2     
Impacted Files Coverage Δ
init.go 67.70% <65.51%> (+8.80%) :arrow_up:
add.go 77.61% <0.00%> (ø)
pkg/commands/format.go 0.00% <0.00%> (ø)
git.go 70.00% <0.00%> (+1.57%) :arrow_up:
files.go 38.88% <0.00%> (+7.63%) :arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

codecov[bot] avatar Jul 17 '22 23:07 codecov[bot]

The ci commit is a throw away. I just wanted to demonstrate this pull requests passes the ci checks.

masonkatz avatar Jul 18 '22 00:07 masonkatz

lgtm, can you rebase @masonkatz

cc/ @djgilcrease as well

caarlos0 avatar Aug 16 '22 01:08 caarlos0

Will never once the conflicts are resolved

djgilcrease avatar Aug 16 '22 02:08 djgilcrease

Rebased

It looks like the ci is in flux right now -- see the linter failing on code outside of this commit.

masonkatz avatar Aug 16 '22 19:08 masonkatz