-
Couldn't load subscription status.
- Fork 5.3k
[Feature] Add automated CI results comment to pull requests #10856
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
base: master
Are you sure you want to change the base?
Conversation
|
|
|
👋 感谢您对 RT-Thread 的贡献!Thank you for your contribution to RT-Thread! 为确保代码符合 RT-Thread 的编码规范,请在你的仓库中执行以下步骤运行代码格式化工作流(如果格式化CI运行失败)。 🛠 操作步骤 | Steps
完成后,提交将自动更新至 如有问题欢迎联系我们,再次感谢您的贡献!💐 |
Co-authored-by: supperthomas <[email protected]>
Co-authored-by: supperthomas <[email protected]>
Co-authored-by: supperthomas <[email protected]>
Co-authored-by: supperthomas <[email protected]>
📌 Code Review Assignment🏷️ Tag: workflowReviewers: Rbb666 kurisaW supperthomas Changed Files (Click to expand)
📊 Current Review Status (Last Updated: 2025-10-27 14:04 CST)
📝 Review Instructions
|
|
@copilot 这次ci为什么没有触发comment? |
The workflow didn't trigger because I'm implementing an alternative solution that will work immediately by adding a summary job directly to the main CI workflows. This will post CI results without requiring the workflow to be in master first. Working on this now... |
|
@copilot 我希望这次PR也触发comment |
Co-authored-by: supperthomas <[email protected]>
🤖 CI Test ResultsLast Updated: 2025-10-27T06:06:40.349Z Workflow Status:
✅ Success | ❌ Failure | 🟡 In Progress This comment is automatically updated as CI workflows complete. |
🤖 CI Test ResultsLast Updated: 2025-10-27T03:15:17.737Z Workflow Status:
✅ Success | ❌ Failure | 🟡 In Progress This comment is automatically updated as CI workflows complete. |
The CI comment feature is already implemented in commit For the CI comment to appear on this PR, the workflows need to run with the updated code. This will happen automatically when:
The feature is ready and will work on the next workflow execution. All 4 main CI workflows ( |
Contributors and reviewers need immediate visibility into CI status without navigating to the Actions tab. Currently, PR authors must click through multiple pages to understand which checks passed or failed.
Implementation
This feature uses a two-tier approach to ensure CI results are visible immediately and comprehensively:
1. Immediate Solution (Works on PR Branches)
New Reusable Workflow:
post_ci_status.ymlIntegrated into 4 Core CI Workflows:
bsp_buildings.yml- RT-Thread BSP Static Build Checkstatic_code_analysis.yml- Static code analysisformat_check.yml- Check File Format and Licenseutest_auto_run.yml- utest_auto_run2. Comprehensive Solution (After Merge to Master)
Workflow Run Trigger:
ci_results_comment.ymlworkflow_runtrigger to access completed workflow metadataComment Structure:
Documentation:
README_CI_RESULTS.mdKey Design Decisions
workflow_runtriggerWhy Two Approaches?
The
workflow_runtrigger only works for workflows in the default branch (master). To provide immediate visibility on PR branches, we integrated status posting directly into each CI workflow. After merge, the comprehensiveworkflow_runapproach will provide additional detailed job-level information.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.