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
Refactor changelog #2839
Refactor changelog #2839
Conversation
docs/changelog/v1.5.2-incubating.md
Outdated
## Changelog for Apache Kyuubi(Incubating) v1.5.2-incubating | ||
- [Fix changelog nit](https://github.com/apache/incubator-kyuubi/commit/62b51aa3) | ||
- [[KYUUBI #2746][INFRA][1.5] Improve NOTICE of binary release](https://github.com/apache/incubator-kyuubi/commit/35a4c488) | ||
- [[KYUUBI-2422] Wrap close session with try-finally (#2836)](https://github.com/apache/incubator-kyuubi/commit/cbca761a) |
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.
this line crashes too
build/release/generate_changelog.sh
Outdated
@@ -49,6 +49,6 @@ if [ "${LAST_RELEASE_TAG}" = "-h" ]; then | |||
fi | |||
|
|||
## Add title for changelog doc. | |||
echo "## Changelog for Apache Incubating Kyuubi $RELEASE_VERSION " > $CHANGELOG_PATH | |||
echo "## Changelog for Apache Kyuubi(Incubating) $RELEASE_VERSION " > $CHANGELOG_PATH | |||
## Append well-formatted git log to changelog file. | |||
git log --pretty="- [%s]($KYUUBI_GITHUB_COMMIT_URL%h) " $LAST_RELEASE_TAG..$CURRENT_RELEASE_TAG | grep -v "\[RELEASE\]" >> $CHANGELOG_PATH |
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.
let's remove the list-item marker -
@yaooqinn refactor the changelog |
you can break this PR into
|
build/release/generate_changelog.sh
Outdated
git log --pretty="[%s]($KYUUBI_GITHUB_COMMIT_URL%h)%n" $LAST_RELEASE_TAG..$CURRENT_RELEASE_TAG | grep -v "\[RELEASE\]" >> $CHANGELOG_PATH |
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.
%n is also unnecessary
@yaooqinn reverted the 1.5.2 changelog |
docs/changelog/v1.5.1-incubating.md
Outdated
- [[KYUUBI #2134] Respect Spark bundled log4j in extension modules](https://github.com/apache/incubator-kyuubi/commit/30dc84b) | ||
- [[KYUUBI #2150] [DOCS] Fix Getting Started With Kyuubi on Kubernetes](https://github.com/apache/incubator-kyuubi/commit/e232a83) | ||
## Changelog for Apache Kyuubi(Incubating) v1.5.1-incubating | ||
[[KYUUBI #2354] Fix NPE in process builder log capture thread](https://github.com/apache/incubator-kyuubi/commit/5e76334e) |
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.
you should not remove 2 tailing spaces
LGTM except the pr desc |
@yaooqinn updated the description |
thanks, merging to master |
Why are the changes needed?
refactor the changelog script to fix:
How was this patch tested?
Add some test cases that check the changes thoroughly including negative and positive cases if possible
Add screenshots for manual tests if appropriate
Run test locally before make a pull request