lalitkapoor / github-changes

Generate a changelog based on merged pull requests or commit messages
https://lalitkapoor.github.io/github-changes/
MIT License
676 stars 52 forks source link

--only-pulls gives different output than --data pulls #65

Closed darkweasel77 closed 7 years ago

darkweasel77 commented 7 years ago

We would like our changelog generated to use the PR title. the deprecated -d pulls did that. Is there a way to do that?

with deprecated -d pulls

> cat CHANGELOG.md
## Change Log

### 1.0.2 (2017/01/26 17:55 +00:00)
- [#6](https://git.target.com/StoreDataMovement/DuckPuppy/pull/6) Pull in properties differently (@AlexBrick)
- [#5](https://git.target.com/StoreDataMovement/DuckPuppy/pull/5) SDM-1745 : Update with StringBuilders for (almost) all (@GeoffThomas)
- [#4](https://git.target.com/StoreDataMovement/DuckPuppy/pull/4) SDM-1745-changed int to big int and return codes (@SenthilKumar)

### 1.0.1 (2017/01/18 19:04 +00:00)
- [#3](https://git.target.com/StoreDataMovement/DuckPuppy/pull/3) SDM  1830 Run as api (@AlexBrick)
- [#1](https://git.target.com/StoreDataMovement/DuckPuppy/pull/1) Sdm coupon parser (@JohnRowe)%

with --only-pulls

> cat CHANGELOG.md
## Change Log

### 1.0.2 (2017/01/26 17:55 +00:00)
- [#6](https://git.target.com/StoreDataMovement/DuckPuppy/pull/6) Merge pull request #6 from StoreDataMovement/UpdatePropertiesStuff (@AlexBrick)
- [#5](https://git.target.com/StoreDataMovement/DuckPuppy/pull/5) Merge pull request #5 from StoreDataMovement/SDM-1745-Coupon-Submitter (@GeoffThomas, @SenthilKumar)
- [#4](https://git.target.com/StoreDataMovement/DuckPuppy/pull/4) Merge pull request #4 from StoreDataMovement/SDM-1745-Coupon-Submitter (@SenthilKumar, @GeoffThomas)

### 1.0.1 (2017/01/18 19:04 +00:00)
- [#3](https://git.target.com/StoreDataMovement/DuckPuppy/pull/3) Merge pull request #3 from StoreDataMovement/RunAsAPI (@NathanGaida, @AlexBrick, @PhilipBreczinski)
- [#1](https://git.target.com/StoreDataMovement/DuckPuppy/pull/1) Merge pull request #1 from StoreDataMovement/SDM-COUPON-Parser (@JohnRowe)%

with --only-pulls --use-commit-body

> cat CHANGELOG.md
## Change Log

### 1.0.2 (2017/01/26 17:55 +00:00)
- [#6](https://git.target.com/StoreDataMovement/DuckPuppy/pull/6) merged by Jenkins (@AlexBrick)
- [#5](https://git.target.com/StoreDataMovement/DuckPuppy/pull/5) merged by Jenkins (@GeoffThomas, @SenthilKumar)
- [#4](https://git.target.com/StoreDataMovement/DuckPuppy/pull/4) merged by Jenkins (@SenthilKumar, @GeoffThomas)

### 1.0.1 (2017/01/18 19:04 +00:00)
- [#3](https://git.target.com/StoreDataMovement/DuckPuppy/pull/3) SDM  1830 Run as api (@NathanGaida, @AlexBrick, @PhilipBreczinski)
- [#1](https://git.target.com/StoreDataMovement/DuckPuppy/pull/1) Sdm coupon parser (@JohnRowe)%
lalitkapoor commented 7 years ago

@darkweasel77 I'm actually going to un-deprecate the --data pulls because of it's support for handling various merge strategies. Please feel free to continue to use -d pulls.

Btw, looks like Jenkins is using a custom commit message than when merging through the github ui