Skip to content

Fix milestone lookup in hotfix workflow#4148

Merged
paulmedynski merged 1 commit intomainfrom
dev/paul/hotfix-action
Apr 7, 2026
Merged

Fix milestone lookup in hotfix workflow#4148
paulmedynski merged 1 commit intomainfrom
dev/paul/hotfix-action

Conversation

@paulmedynski
Copy link
Copy Markdown
Contributor

Description

Fixed the milestone lookup during the hotfix cherry-pick workflow. The correct milestone, based on the "Hotfix X.Y.Z" label, should now be assigned to the new backport PR.

@paulmedynski paulmedynski added this to the 7.1.0-preview1 milestone Apr 7, 2026
@paulmedynski paulmedynski added the Area\Engineering Use this for issues that are targeted for changes in the 'eng' folder or build systems. label Apr 7, 2026
Copilot AI review requested due to automatic review settings April 7, 2026 10:22
@paulmedynski paulmedynski requested a review from a team as a code owner April 7, 2026 10:22
@github-project-automation github-project-automation Bot moved this to To triage in SqlClient Board Apr 7, 2026
@paulmedynski paulmedynski moved this from To triage to In review in SqlClient Board Apr 7, 2026
Copy link
Copy Markdown
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 fixes milestone lookup in the hotfix cherry-pick automation so that the workflow reliably reads existing milestones (instead of risking an unintended create-call behavior) and can correctly assign the matching milestone to the backport PR.

Changes:

  • Update gh api milestone lookup to explicitly use --method GET during pagination/field usage.
  • Add Bats assertions to ensure milestone lookup uses GET.
  • Bump the hotfix workflow’s checkout action to actions/checkout@v5.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
.github/workflows/cherry-pick-hotfix.yml Updates workflow checkout steps to actions/checkout@v5.
.github/scripts/tests/cherry-pick-to-release.bats Adds test assertions that milestone lookup calls gh api ... --method GET.
.github/scripts/cherry-pick-to-release.sh Forces milestone lookup to use GET to avoid incorrect endpoint/method behavior.

@paulmedynski paulmedynski merged commit 27999d9 into main Apr 7, 2026
13 checks passed
@paulmedynski paulmedynski deleted the dev/paul/hotfix-action branch April 7, 2026 15:57
@github-project-automation github-project-automation Bot moved this from In review to Done in SqlClient Board Apr 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area\Engineering Use this for issues that are targeted for changes in the 'eng' folder or build systems.

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants