-
Notifications
You must be signed in to change notification settings - Fork 37
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
Feature/add changelog link #16
base: master
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the MR :D
@@ -211,5 +213,47 @@ describe("ChangelogLib lib", () => { | |||
"- test1 [#1](http://gitlab.example.com/my-group/my-project/merge_requests/1) ([admin](https://gitlab.example.com/admin))\n"); | |||
}); | |||
}); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we have a test case with fullChangeLogLink please?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added, but as mentioned in the description, I'm unable to test (or could you explain how to test?)
@jk1z Thanks for your review! I left some comments on it. |
Hi, I have made small refactoring for this and added test for the first tag. Because there should be no compare link on the first tag. cookielab@f0612cb |
The documentation of the original repo links to this output example
https://github.com/github-changelog-generator/Github-Changelog-Generator/blob/master/CHANGELOG.md
That example includes a "Full Changelog" link, which basically uses the GitHub compare functionality.
This PR adds the ability to include a Full Changelog for GitLab as well.
Note to maintainer:
I've added some tests to cover this functionality, but didn't know how to run those tests. So please make sure to check those extensively.