Skip to content

[projmgr] Add memory.pname into input target type node - #2541

Merged
brondani merged 1 commit into
mainfrom
projmgr-add-memory-pname
Jul 27, 2026
Merged

[projmgr] Add memory.pname into input target type node#2541
brondani merged 1 commit into
mainfrom
projmgr-add-memory-pname

Conversation

@brondani

@brondani brondani commented Jul 27, 2026

Copy link
Copy Markdown
Collaborator

Fixes

Changes

  • Add memory.pname into schema and into parser
  • Extend TestRunDebugMulticore test case.

Checklist

  • 🤖 This change is covered by unit tests (if applicable).
  • 🤹 Manual testing has been performed (if necessary).
  • 🛡️ Security impacts have been considered (if relevant).
  • 📖 Documentation updates are complete (if required).
  • 🧠 Third-party dependencies and TPIP updated (if required).

@github-actions

Copy link
Copy Markdown

Test Results

  2 files   -    37   14 suites   - 107   13m 37s ⏱️ - 4m 41s
476 tests  -   451  476 ✅  - 434  0 💤  - 17  0 ❌ ±0 
952 runs   - 1 041  952 ✅  - 990  0 💤  - 51  0 ❌ ±0 

Results for commit ee361f7. ± Comparison against base commit b0c5f28.

This pull request removes 451 tests.
AuxCmdTests ‑ MkdirCmdTest
AuxCmdTests ‑ RmdirCmdTest
AuxCmdTests ‑ TouchCmdTest
BuildSystemGeneratorTests ‑ GenAuditFile
BuildSystemGeneratorTests ‑ GenAuditFile_WithOut_Existing_Audit_File
BuildSystemGeneratorTests ‑ GenAuditFile_With_Existing_Audit_File
BuildSystemGeneratorTests ‑ GetString
BuildSystemGeneratorTests ‑ StrConv
BuildSystemGeneratorTests ‑ StrNorm
CBuildGCCTests ‑ Asm
…

@brondani
brondani marked this pull request as ready for review July 27, 2026 08:14
@brondani
brondani requested a review from soumeh01 July 27, 2026 08:14
@codecov

codecov Bot commented Jul 27, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 65.37%. Comparing base (b0c5f28) to head (ee361f7).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2541      +/-   ##
==========================================
+ Coverage   65.36%   65.37%   +0.01%     
==========================================
  Files         147      147              
  Lines       26803    26805       +2     
  Branches    16240    16242       +2     
==========================================
+ Hits        17521    17525       +4     
  Misses       7075     7075              
+ Partials     2207     2205       -2     
Flag Coverage Δ
projmgr-cov 87.99% <100.00%> (+0.02%) ⬆️

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

Files with missing lines Coverage Δ
tools/projmgr/include/ProjMgrParser.h 100.00% <ø> (ø)
tools/projmgr/src/ProjMgrRunDebug.cpp 87.80% <100.00%> (+0.22%) ⬆️
tools/projmgr/src/ProjMgrYamlParser.cpp 84.93% <100.00%> (+0.01%) ⬆️

... and 1 file with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@soumeh01 soumeh01 left a comment

Copy link
Copy Markdown
Contributor

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 c3edeca into main Jul 27, 2026
30 checks passed
@brondani
brondani deleted the projmgr-add-memory-pname branch July 27, 2026 08:25
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