CI: MemBrowse integration#11478
Conversation
|
|
|
👋 感谢您对 RT-Thread 的贡献！Thank you for your contribution to RT-Thread! 为确保代码符合 RT-Thread 的编码规范，请在你的仓库中执行以下步骤运行代码格式化工作流（如果格式化CI运行失败）。 🛠 操作步骤 | Steps
完成后，提交将自动更新至 如有问题欢迎联系我们，再次感谢您的贡献！💐 |
📌 Code Review Assignment🏷️ Tag: workflowReviewers: Rbb666 kurisaW supperthomas Changed Files (Click to expand)
📊 Current Review Status (Last Updated: 2026-06-13 21:05 CST)
📝 Review Instructions
|
michael-membrowse
force-pushed
the
master
branch
from
670538c to
1a82239
Compare
There was a problem hiding this comment.
Pull request overview
This PR integrates MemBrowse into the RT-Thread GitHub Actions CI so that selected BSP builds can generate memory usage reports and publish/comment results (plus a README badge).
本次 PR 将 MemBrowse 集成到 RT-Thread 的 GitHub Actions CI 中，使部分 BSP 构建可生成内存使用报告并发布/评论结果（并在 README 增加徽章）。
Changes / 变更：
- Add a MemBrowse report workflow that builds a matrix of BSP targets on PRs and pushes to
master, then uploads artifacts for MemBrowse analysis.
新增 MemBrowse 报告工作流：在 PR 与推送到master时按矩阵构建 BSP，并上传给 MemBrowse 分析。 - Add workflows for onboarding historical commits and for posting a combined PR comment from report results.
新增历史数据导入（onboard）与 PR 评论汇总工作流。 - Add a target definition file (
membrowse-targets.json) and a MemBrowse badge inREADME.md.
新增目标配置文件（membrowse-targets.json）以及 README 徽章。
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| README.md | Adds MemBrowse badge linking to the public dashboard. |
| .github/workflows/membrowse-report.yml | Builds selected BSP targets and runs MemBrowse analysis on PR/push. |
| .github/workflows/membrowse-onboard.yml | Manual workflow to upload historical MemBrowse reports. |
| .github/workflows/membrowse-comment.yml | Posts MemBrowse summary comment on PRs after report workflow completes. |
| .github/membrowse-targets.json | Defines the CI build/analysis target matrix (BSP paths, toolchains, artifacts). |
| [](https://github.com/RT-Thread/rt-thread/pulls) | ||
| [](https://github.com/RT-Thread/rt-thread/pulls) | ||
| [](https://github.com/RT-Thread/rt-thread/actions/workflows/bsp_buildings.yml) | ||
| [](https://membrowse.com/public/RT-Thread/rt-thread) |
| - name: Run MemBrowse analysis | ||
| uses: membrowse/membrowse-action@v1 | ||
| with: | ||
| target_name: ${{ matrix.target_name }} | ||
| elf: ${{ matrix.elf }} | ||
| ld: ${{ matrix.ld }} | ||
| map_file: ${{ matrix.map_file }} | ||
| linker_vars: ${{ matrix.linker_vars }} | ||
| api_key: ${{ secrets.MEMBROWSE_API_KEY }} | ||
| api_url: ${{ vars.MEMBROWSE_API_URL }} | ||
| verbose: INFO | ||
| # Uncomment to allow CI to pass even when memory budgets are exceeded | ||
| # dont_fail_on_alerts: true |
| - name: Post combined PR comment | ||
| if: ${{ env.MEMBROWSE_API_KEY != '' }} | ||
| uses: membrowse/membrowse-action/comment-action@v1 | ||
| with: | ||
| api_key: ${{ secrets.MEMBROWSE_API_KEY }} | ||
| commit: ${{ github.event.workflow_run.head_sha }} | ||
| env: | ||
| MEMBROWSE_API_KEY: ${{ secrets.MEMBROWSE_API_KEY }} | ||
| GH_TOKEN: ${{ secrets.GITHUB_TOKEN }} |
拉取/合并请求描述：(PR description)
[
为什么提交这份PR (why to submit this PR)
This adds integration of MemBrowse into the CI.
As presented in the RT-Thread Global Conference.
Here is a dashboard from a forked repo: https://membrowse.com/public/michael-membrowse/rt-thread
Now for every change in a PR and push to dev a memory report will be generated and sent to membrowse.com
To configure the setup:
你的解决方案是什么 (what is your solution)
Full docs are available at docs.membrowse.com.
Happy to help if any questions come up during integration.
]
当前拉取/合并请求的状态 Intent for your PR
必须选择一项 Choose one (Mandatory):
代码质量 Code Quality：
我在这个拉取/合并请求中已经考虑了 As part of this pull request, I've considered the following:
#if 0代码，不包含已经被注释了的代码 All redundant code is removed and cleaned up