Skip to content

[csolution-rpc] Expose evaluated dynamic access sequences#2388

Merged
brondani merged 1 commit into
Open-CMSIS-Pack:mainfrom
ARM-software:outbound/projmgr-expose-dynamic-sequences
Mar 6, 2026
Merged

[csolution-rpc] Expose evaluated dynamic access sequences#2388
brondani merged 1 commit into
Open-CMSIS-Pack:mainfrom
ARM-software:outbound/projmgr-expose-dynamic-sequences

Conversation

@grasci-arm

@grasci-arm grasci-arm commented Mar 6, 2026

Copy link
Copy Markdown
Collaborator

Store absolute paths of evaluated dynamic access sequences such as OutDir().
Expose them via RPC methods GetVariables and GetContextInfo.
Background support for ARM-software/vscode-cmsis-csolution#510

@github-actions

github-actions Bot commented Mar 6, 2026

Copy link
Copy Markdown

Test Results

    3 files     21 suites   14m 25s ⏱️
  450 tests   450 ✅ 0 💤 0 ❌
1 350 runs  1 350 ✅ 0 💤 0 ❌

Results for commit 341d03d.

@brondani brondani merged commit 8f7ac41 into Open-CMSIS-Pack:main Mar 6, 2026
33 checks passed
@brondani brondani deleted the outbound/projmgr-expose-dynamic-sequences branch March 6, 2026 15:21
@codecov

codecov Bot commented Mar 6, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 58.82353% with 7 lines in your changes missing coverage. Please review.
✅ Project coverage is 64.56%. Comparing base (fb3e895) to head (341d03d).
⚠️ Report is 2 commits behind head on main.

Files with missing lines Patch % Lines
tools/projmgr/src/ProjMgrWorker.cpp 53.33% 1 Missing and 6 partials ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2388   +/-   ##
=======================================
  Coverage   64.55%   64.56%           
=======================================
  Files         145      145           
  Lines       26085    26090    +5     
  Branches    15733    15738    +5     
=======================================
+ Hits        16839    16844    +5     
  Misses       7089     7089           
  Partials     2157     2157           
Flag Coverage Δ
projmgr-cov 87.46% <58.82%> (+<0.01%) ⬆️

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

Files with missing lines Coverage Δ
tools/projmgr/include/ProjMgrWorker.h 83.33% <ø> (ø)
tools/projmgr/src/ProjMgrRpcServer.cpp 83.50% <100.00%> (+0.05%) ⬆️
tools/projmgr/src/ProjMgrWorker.cpp 87.98% <53.33%> (+<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