Thanks to visit codestin.com
Credit goes to github.com

Skip to content

The pull request label check wasn't getting run when a draft PR was published #49546

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

Merged
merged 2 commits into from
Jul 1, 2025

Conversation

marcpopMSFT
Copy link
Member

@marcpopMSFT marcpopMSFT commented Jun 24, 2025

copilot creates it's PRs in draft mode and then we publish after discussion. I noticed that the branch lockdown check wouldn't end up getting run and I'd have to force it to run. This should fix that I believe.

@Copilot Copilot AI review requested due to automatic review settings June 24, 2025 22:19
Copy link
Contributor

@Copilot 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

Ensures the PR analysis workflow runs when a draft PR is marked ready for review by adding the ready_for_review event.

  • Added a pull_request trigger for the ready_for_review type in the workflow.

Copy link
Member

@MiYanni MiYanni left a comment

Choose a reason for hiding this comment

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

All I did was read the document on pull_request_target. I learned that this was chosen because of this sentence:

This event runs in the context of the base of the pull request, rather than in the context of the merge commit, as the pull_request event does.

So, there's no reason to use/add pull_request.

@baronfel baronfel merged commit e0034a5 into main Jul 1, 2025
26 checks passed
@baronfel baronfel deleted the marcpopMSFT-fixdraftPRpublishcheck branch July 1, 2025 20:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants