Skip to content

Conversation

sjmonson
Copy link
Collaborator

@sjmonson sjmonson commented Jun 17, 2025

Run the development workflow on the pull_request_target trigger which runs the workflow from the scope of the target branch rather than the merge branch. This should solve the permission failure in the build job.

@sjmonson sjmonson changed the title Only build if the PR branch is local WIP: Only build if the PR branch is local Jun 17, 2025
@sjmonson sjmonson force-pushed the fix/build_external_failure branch from db914fa to d72f086 Compare June 17, 2025 20:13
@sjmonson sjmonson changed the title WIP: Only build if the PR branch is local Only build if the PR branch is local Jun 17, 2025
@sjmonson sjmonson force-pushed the fix/build_external_failure branch from d72f086 to 23221ef Compare June 17, 2025 20:44
@sjmonson sjmonson changed the title Only build if the PR branch is local WIP: Only build if the PR branch is local Jun 17, 2025
@sjmonson sjmonson force-pushed the fix/build_external_failure branch 8 times, most recently from 220a463 to 370c142 Compare June 17, 2025 21:26
@sjmonson sjmonson force-pushed the fix/build_external_failure branch from 370c142 to e5972be Compare June 17, 2025 21:26
@sjmonson
Copy link
Collaborator Author

sjmonson commented Jun 17, 2025

I have spent way too much time trying to figure out why the checks will not trigger and I think the answer is that this PR must be merged into main and then can be tested from the next PR. (ref)

@sjmonson sjmonson changed the title WIP: Only build if the PR branch is local Run development workflow from base repo scope Jun 17, 2025
@sjmonson sjmonson changed the title Run development workflow from base repo scope Run development workflow from target branch scope Jun 17, 2025
@sjmonson sjmonson requested review from Copilot and markurtz June 17, 2025 21:37
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

This PR updates the development workflow trigger to use pull_request_target so that the workflow runs from the target branch scope, addressing the permission failure in the build job.

  • Updated workflow trigger from "pull_request" to "pull_request_target"
  • Adjusted event types to maintain appropriate workflow behavior

@markurtz markurtz merged commit fd1bf3c into vllm-project:main Jun 20, 2025
sjmonson added a commit that referenced this pull request Jul 10, 2025
sjmonson added a commit that referenced this pull request Jul 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants