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

Support for external generators in RTE Model (continue) #1310

Merged
merged 1 commit into from
Feb 2, 2024

Conversation

grasci-arm
Copy link
Collaborator

Support for external generators in RTE Model

remove obsolete unused code

Support for external generators in RTE Model

remove obsolete unused code
Copy link

codecov bot commented Feb 2, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (8b3b1eb) 59.06% compared to head (8b48e32) 59.12%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1310      +/-   ##
==========================================
+ Coverage   59.06%   59.12%   +0.05%     
==========================================
  Files         117      117              
  Lines       23438    23415      -23     
  Branches    13070    13058      -12     
==========================================
  Hits        13844    13844              
+ Misses       7347     7324      -23     
  Partials     2247     2247              
Flag Coverage Δ
projmgr-cov 84.66% <ø> (+0.35%) ⬆️

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

Files Coverage Δ
tools/projmgr/src/ProjMgrParser.cpp 100.00% <ø> (+12.12%) ⬆️
tools/projmgr/src/ProjMgrYamlParser.cpp 77.07% <ø> (+2.04%) ⬆️

Copy link

github-actions bot commented Feb 2, 2024

Test Results

  3 files   18 suites   3m 59s ⏱️
306 tests 306 ✅ 0 💤 0 ❌
918 runs  918 ✅ 0 💤 0 ❌

Results for commit 8b48e32.

@edriouk edriouk merged commit b949d50 into Open-CMSIS-Pack:main Feb 2, 2024
33 checks passed
@edriouk edriouk deleted the outbound/extGenRte1 branch February 2, 2024 16:57
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