Skip to content

Add ?pr=<pr_number> parameter to workflow URL in resolve.yml#18686

Merged
harupy merged 3 commits intomasterfrom
copilot/update-workflow-url-with-pr
Nov 5, 2025
Merged

Add ?pr=<pr_number> parameter to workflow URL in resolve.yml#18686
harupy merged 3 commits intomasterfrom
copilot/update-workflow-url-with-pr

Conversation

Copy link
Contributor

Copilot AI commented Nov 5, 2025 •
edited by github-actions bot
Loading

🛠 DevTools 🛠

Open in GitHub Codespaces

Install mlflow from this PR

# mlflow
pip install git+https://github.com/mlflow/mlflow.git@refs/pull/18686/merge
# mlflow-skinny
pip install git+https://github.com/mlflow/mlflow.git@refs/pull/18686/merge#subdirectory=libs/skinny

For Databricks, use the following command:

%sh curl -LsSf https://raw.githubusercontent.com/mlflow/mlflow/HEAD/dev/install-skinny.sh | sh -s pull/18686/merge

When GitHub detects the ?pr=<pr_number> parameter in a workflow run URL, it displays a "Back to pull request #..." button for convenient navigation.

Changes

  • Updated the workflow URL construction in .github/workflows/resolve.yml to append ?pr=${context.issue.number}, matching the existing pattern in review.yml
- const workflowUrl = `https://github.com/${context.repo.owner}/${context.repo.repo}/actions/runs/${context.runId}`;
+ const workflowUrl = `https://github.com/${context.repo.owner}/${context.repo.repo}/actions/runs/${context.runId}?pr=${context.issue.number}`;
Original prompt

Append ?pr=<pr_number> to workflowUrl in .github/workflows/resolve.yml. When GitHub detects this parameter, it displays a convenient "Back to pull request #..." button on the workflow run page, improving navigation between the workflow run and the original PR. See .github/workflows/review.yml for an example.

const workflowUrl = `https://github.com/${context.repo.owner}/${context.repo.repo}/actions/runs/${context.runId}`;

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

@harupy
Add ?pr parameter to workflowUrl in resolve.yml for better navigation
0a63fd5 
Co-authored-by: harupy <17039389+harupy@users.noreply.github.com>
Copilot AI changed the title [WIP] Update workflowUrl to include PR number Add ?pr=<pr_number> parameter to workflow URL in resolve.yml Nov 5, 2025
Copilot AI requested a review from harupy November 5, 2025 11:22
@harupy harupy marked this pull request as ready for review November 5, 2025 11:22
@harupy harupy added the rn/none List under Small Changes in Changelogs. label Nov 5, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Nov 5, 2025

@Copilot Thank you for the contribution! Could you fix the following issue(s)?

⚠ Invalid PR template

This PR does not appear to have been filed using the MLflow PR template. Please copy the PR template from here and fill it out.

@harupy harupy added this pull request to the merge queue Nov 5, 2025
Merged via the queue into master with commit 035184e Nov 5, 2025
46 of 48 checks passed
@harupy harupy deleted the copilot/update-workflow-url-with-pr branch November 5, 2025 12:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@harupy harupy harupy approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@harupy harupy

Copilot code review Copilot

Labels

rn/none List under Small Changes in Changelogs.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@harupy