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

packaging: release notes: Don't show shortlist by default, and add survey link #9076

Merged
Merged
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
21 changes: 21 additions & 0 deletions tools/packaging/release/release-notes.sh
Original file line number Diff line number Diff line change
Expand Up @@ -75,9 +75,19 @@ changes() {
echo "**FIXME - message this section by hand to produce a summary please**"

echo "### Shortlog"

echo "<details>"
echo "<summary>Click the icon to show the list of commits included in this release</summary>"

# XXX: Essential to have at least one blank line here. It forces
# GitHub to show each commit on a separate line.
echo

for cr in $(git log --merges "${previous_release}".."${new_release}" | grep 'Merge:' | awk '{print $2".."$3}'); do
git log --oneline "$cr"
done

echo "</details>"
}

print_release_notes() {
Expand All @@ -94,6 +104,17 @@ EOF
## ${repo} Changes
$(changes)

## Survey

Please take the Kata Containers survey:

- https://openinfrafoundation.formstack.com/forms/kata_containers_user_survey

This will help the Kata Containers community understand:

- how you use Kata Containers
- what features and improvements you would like to see in Kata Containers

EOF
popd >>/dev/null
rm -rf "${tmp_dir}/${repo}"
Expand Down