Skip to content

ci(framework): Fix framework/dev/get-latest-changelog.sh to retrieve changelog correctly for draft releases#6739

Merged
danieljanes merged 3 commits intomainfrom
fix-get-latest-changelog
Mar 10, 2026
Merged

ci(framework): Fix framework/dev/get-latest-changelog.sh to retrieve changelog correctly for draft releases#6739
danieljanes merged 3 commits intomainfrom
fix-get-latest-changelog

Conversation

@panh99
Copy link
Contributor

@panh99 panh99 commented Mar 10, 2026

No description provided.

Copilot AI review requested due to automatic review settings March 10, 2026 12:51
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the framework draft-release helper script to extract the latest release notes from the documentation changelog in a way that works even when the Git tag naming (e.g., framework-X.Y.Z) doesn’t match the changelog headings (e.g., ## vX.Y.Z).

Changes:

  • Switch changelog section detection from git tag ordering to parsing the first two ## vX.Y.Z ... headings in framework/docs/source/ref-changelog.md.
  • Add stricter shell options (set -euo pipefail) and an explicit error when versions can’t be determined.
  • Tighten the awk start/end match to anchor on changelog headings.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

@github-actions github-actions bot added the Maintainer Used to determine what PRs (mainly) come from Flower maintainers. label Mar 10, 2026
@danieljanes danieljanes merged commit 5a741bb into main Mar 10, 2026
70 checks passed
@danieljanes danieljanes deleted the fix-get-latest-changelog branch March 10, 2026 13:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Maintainer Used to determine what PRs (mainly) come from Flower maintainers.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants