Enhance publish-docs workflow: Include triggering upstream PR details in autogenerated docs PRs#11159
Enhance publish-docs workflow: Include triggering upstream PR details in autogenerated docs PRs#11159
Conversation
Unit Tests 2 files + 1 415 suites +85 6m 38s ⏱️ -20s Results for commit 98961a6. ± Comparison against base commit c41f296. This pull request removes 1 and adds 6 tests. Note that renamed tests count towards both.♻️ This comment has been updated with latest results. |
There was a problem hiding this comment.
Pull request overview
Updates the publish-docs GitHub Actions workflow so autogenerated PRs to the docs repo include richer context about what triggered the docs generation (merged PR vs direct commit vs manual dispatch), improving traceability back to Radius changes.
Changes:
- Add a workflow step to infer trigger context from the triggering commit message and build a formatted PR title/body.
- Use the generated title/body outputs when creating the docs pull request.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #11159 +/- ##
==========================================
- Coverage 51.42% 51.38% -0.05%
==========================================
Files 699 699
Lines 55723 44111 -11612
==========================================
- Hits 28654 22665 -5989
+ Misses 24901 19278 -5623
Partials 2168 2168 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
e6be2d1 to
6fea2d2
Compare
Dependency Review SummaryThe full dependency review summary was too large to display here (1461KB, limit is 1024KB). Please download the artifact named "dependency-review-summary" to view the complete report. |
Signed-off-by: Will Tsai <28876888+willtsai@users.noreply.github.com>
d43f899 to
c3637c6
Compare
DariuszPorowski
left a comment
There was a problem hiding this comment.
I do not want to block so far, so let move... idea for the future - we can reduce complexity by using https://github.com/radius-project/.github/blob/820618e948e92d911ba3ac46a80e8169d5e3a4d4/.github/workflows/sync.yml#L55 and avoid maintaining own custom script.
Radius functional test overviewClick here to see the test run details
Test Status⌛ Building Radius and pushing container images for functional tests... |
Description
The
publish-docsworkflow generates PRs against the docs repository without context about the triggering upstream change. This makes it difficult for reviewers to trace documentation updates back to source code changes.Changes
Added context extraction step that detects trigger type and formats rich PR descriptions:
Merged PRs - Extracts PR number and title from squash merge commit messages using pattern
\(#([0-9]+)\)$Update auto-generated documentation (PR #1234)Direct commits - Captures commit metadata for non-PR changes
Update auto-generated documentation (main)Manual triggers - Identifies workflow_dispatch invocations
Update auto-generated documentation (manual trigger)Implementation:
.github/scripts/prepare-docs-pr-context.shfor local testability${{ github.ref_name }}to preserve full branch names (e.g.,release/0.32)Files Changed
.github/scripts/prepare-docs-pr-context.sh- Standalone script containing PR context preparation logic.github/workflows/publish-docs.yaml- Simplified to call the script with environment variablesBefore
After
Type of change
Contributor checklist
Please verify that the PR meets the following requirements, where applicable:
Original prompt
💡 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.