Fix push-to-pull-request-branch summary to link commit URL instead of branch URL#6031
Conversation
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
|
🔮 The ancient spirits stir... Smoke Codex awakens to divine this pull request... |
|
📰 BREAKING: Smoke Copilot Playwright is now investigating this pull request. Sources say the story is developing... |
|
💥 WHOOSH! Smoke Claude springs into action on this pull request! [Panel 1 begins...] |
|
📰 BREAKING: Smoke Copilot is now investigating this pull request. Sources say the story is developing... |
|
🎉 Yo ho ho! Changeset Generator found the treasure and completed successfully! ⚓💰 https://github.com/githubnext/gh-aw/tree/copilot/add-context-to-comments |
Smoke Test ResultsLast 2 Merged PRs:
Test Results:
Overall Status: PASS
|
Smoke Test ResultsTests:
Status: FAIL (GitHub API connectivity issue) 📰🔥📋 https://github.com/githubnext/gh-aw/actions/runs/20102297015 fer issue #6031 🗺️ |
|
Smoke Test: Claude Engine - Run 20102297001 Last 2 Merged PRs:
Test Results: Overall Status: PASS (5/6 tests passed)
|
Smoke Test Results✅ Playwright MCP: Page title contains "GitHub" Overall Status: FAIL (1/3 tests failed)
|
|
@copilot also update the code that updates the add-comment comment to display commit url See #6031 (comment) |
|
Latest merged PRs: Add bots field for workflow trigger authorization; Remove COPILOT_CLI_TOKEN environment variable support
|
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
The workflow run summary for
push-to-pull-request-branchwas linking the "Commit" label to the branch tree view (/tree/{branchName}) instead of the commit view (/commit/{commitSha}).Changes
Changed the summary template in
pkg/workflow/js/push_to_pull_request_branch.cjsline 424:The
commitUrlvariable was already correctly defined and used for activation comment updates. This aligns the summary output with that behavior.Original prompt
💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.
Changeset