Skip to content

[projmgr] Extend PLM status messages with parent component ID#2417

Merged
brondani merged 4 commits intoOpen-CMSIS-Pack:mainfrom
jthuangarm:plm-message
Mar 27, 2026
Merged

[projmgr] Extend PLM status messages with parent component ID#2417
brondani merged 4 commits intoOpen-CMSIS-Pack:mainfrom
jthuangarm:plm-message

Conversation

@jthuangarm
Copy link
Copy Markdown
Contributor

@jthuangarm jthuangarm commented Mar 27, 2026

To address #2416,
enhance the clarity of update messages related to configuration file updates by including the originating component's ID in the output.

  • Added a new method GetPartialComponentID(bool withBundle) to the RteFileInstance class, allowing retrieval of a component's partial ID. [1] [2]
  • Updated the messages in ProjMgrWorker::CheckConfigPLMFiles to include the partial component ID when available.

Before changes

warning csolution: file '..../ConfigFilesUpdate/Minor/Device/RteTest_ARMCM3/startup_ARMCM3.c' update suggested; merge content from update file, rename update file to base file and remove previous base file

After changes

warning csolution: update suggested for file '..../ConfigFilesUpdate/Minor/Device/RteTest_ARMCM3/startup_ARMCM3.c' from component 'Device:Startup&RteTest Startup'.
Merge content from update file, rename update file to base file and remove previous base file

@github-actions
Copy link
Copy Markdown

github-actions bot commented Mar 27, 2026

Test Results

 33 files   75 suites   11s ⏱️
172 tests 172 ✅ 0 💤 0 ❌
516 runs  516 ✅ 0 💤 0 ❌

Results for commit e410ec3.

♻️ This comment has been updated with latest results.

@jthuangarm jthuangarm marked this pull request as ready for review March 27, 2026 14:39
@jthuangarm jthuangarm requested a review from brondani March 27, 2026 14:40
Copy link
Copy Markdown
Collaborator

@brondani brondani left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@brondani brondani merged commit 6edc54f into Open-CMSIS-Pack:main Mar 27, 2026
84 checks passed
@codecov
Copy link
Copy Markdown

codecov bot commented Mar 27, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 64.73%. Comparing base (bbe461d) to head (e410ec3).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2417   +/-   ##
=======================================
  Coverage   64.72%   64.73%           
=======================================
  Files         145      145           
  Lines       26141    26142    +1     
  Branches    15777    15778    +1     
=======================================
+ Hits        16921    16922    +1     
  Misses       7066     7066           
  Partials     2154     2154           
Flag Coverage Δ
buildmgr-cov 79.87% <ø> (ø)
packchk-cov 69.73% <ø> (ø)
projmgr-cov 87.76% <100.00%> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
tools/projmgr/src/ProjMgrWorker.cpp 88.20% <100.00%> (+<0.01%) ⬆️
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants