Skip to content
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

[projmgr] Strip build metadata when handling pack versions #1382

Merged

Conversation

grasci-arm
Copy link
Collaborator

@grasci-arm grasci-arm commented Mar 18, 2024

Strip build metadata when handling pack versions.
Throw a warning when specified metadata does not match the one from the loaded pack.
Addresses #1339 (comment)

Copy link

codecov bot commented Mar 18, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 59.20%. Comparing base (5d6dbb2) to head (446f0ff).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1382      +/-   ##
==========================================
+ Coverage   59.18%   59.20%   +0.01%     
==========================================
  Files         117      117              
  Lines       23447    23456       +9     
  Branches    13100    13108       +8     
==========================================
+ Hits        13878    13887       +9     
  Misses       7310     7310              
  Partials     2259     2259              
Flag Coverage Δ
projmgr-cov 84.61% <100.00%> (+0.02%) ⬆️

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

Files Coverage Δ
tools/projmgr/src/ProjMgrGenerator.cpp 89.09% <100.00%> (ø)
tools/projmgr/src/ProjMgrWorker.cpp 84.79% <100.00%> (+0.04%) ⬆️

Copy link

github-actions bot commented Mar 18, 2024

Test Results

 33 files   66 suites   4s ⏱️
153 tests 153 ✅ 0 💤 0 ❌
459 runs  459 ✅ 0 💤 0 ❌

Results for commit 446f0ff.

♻️ This comment has been updated with latest results.

@jkrech jkrech linked an issue Mar 19, 2024 that may be closed by this pull request
Copy link
Member

@jkrech jkrech left a comment

Choose a reason for hiding this comment

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

LGTM -> no meta information in cbuild*.yml files other than selected-by and warning if explicitly set meta-data does not match the meta data of the loaded pdsc file.

@jkrech jkrech merged commit 6105f2a into Open-CMSIS-Pack:main Mar 19, 2024
42 checks passed
@jkrech jkrech deleted the outbound/projmgr-strip-pack-metadata branch March 19, 2024 06:38
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.

Pack version does not include meta-data in cbuild*.yml files
3 participants