-
Notifications
You must be signed in to change notification settings - Fork 40.5k
Changelog not formatted correctly #47805
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
Comments
Hm... I wonder if it's because the second line starts with a backtick? It would be easy to reword/rewrap to change that. The "1. ..." "2. ..." section is also indented too much... I had written it to look right as plain text since that's how it got displayed in the release note field in the PR, but I guess it's supposed to be markdown? |
Yeah, there's a bunch of translations happening here (pandoc) to get HTML formatted email notifications and md for the changelog from the same human curated md source extracted out of the PR descriptions. Release notes really should be much simpler than this. There's no need to add an "Action Required" headline as that happens automatically with the At this point, the release notes can be hand-modified within changelog as needed to correct any formatting issues. |
We need to fix this, but I don't think this is the release blocker. |
[MILESTONENOTIFIER] Milestone Removed Important: priority: Must specify exactly one of [ Removing it from the milestone. |
Issues go stale after 90d of inactivity. Prevent issues from auto-closing with an If this issue is safe to close now please do so with Send feedback to sig-testing, kubernetes/test-infra and/or |
Stale issues rot after 30d of inactivity. If this issue is safe to close now please do so with Send feedback to sig-testing, kubernetes/test-infra and/or fejta. |
Rotten issues close after 30d of inactivity. Send feedback to sig-testing, kubernetes/test-infra and/or fejta. |
The v1.7.0-alpha.4 changelog generated from #39164 PR description has weird format:
cc @danwinship
The text was updated successfully, but these errors were encountered: