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

Do not create RTE_Components.h if no component is selected (#926) #1527

Merged
merged 1 commit into from
May 21, 2024

Conversation

grasci-arm
Copy link
Collaborator

  • Do not create RTE_Components.h if no component is selected

  • [projmgr] Do not emit reference to RTE_Components.h when it does not exist


Co-authored-by: Daniel Brondani daniel.brondani@arm.com

…S-Pack#926)

* Do not create RTE_Components.h if no component is selected

* [projmgr] Do not emit reference to `RTE_Components.h` when it does not exist

---------

Co-authored-by: Daniel Brondani <daniel.brondani@arm.com>
Copy link

codecov bot commented May 21, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 63.36%. Comparing base (21ba587) to head (da13431).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1527   +/-   ##
=======================================
  Coverage   63.36%   63.36%           
=======================================
  Files         162      162           
  Lines       30910    30911    +1     
  Branches    18943    18944    +1     
=======================================
+ Hits        19586    19587    +1     
- Misses       7521     7523    +2     
+ Partials     3803     3801    -2     
Flag Coverage Δ
buildmgr-cov 74.12% <ø> (ø)
packchk-cov 65.54% <ø> (ø)
projmgr-cov 80.46% <100.00%> (+<0.01%) ⬆️

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

Files Coverage Δ
tools/projmgr/src/ProjMgrYamlEmitter.cpp 94.54% <100.00%> (+<0.01%) ⬆️

... and 5 files with indirect coverage changes

Copy link

github-actions bot commented May 21, 2024

Test Results

  7 files   53 suites   4m 50s ⏱️
186 tests 169 ✅ 17 💤 0 ❌
695 runs  627 ✅ 68 💤 0 ❌

Results for commit da13431.

♻️ This comment has been updated with latest results.

@ReinhardKeil
Copy link
Collaborator

I'm not sure that this requirement is correct. There is a define "cmsis_device_header" in RTE_components.h that should be also there when no component is selected

@jkrech jkrech self-requested a review May 21, 2024 14:09
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

@jkrech jkrech merged commit 3a91091 into Open-CMSIS-Pack:main May 21, 2024
84 checks passed
@jkrech jkrech deleted the outbound/NoRteCompH branch May 21, 2024 14:10
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.

4 participants