search

Open-CMSIS-Pack / devtools

Open-CMSIS-Pack development tools - C++
Apache License 2.0
70 stars 51 forks source link

[projmgr] Remove `configurable` endianness from target attributes #1485

Closed grasci-arm closed 2 months ago

grasci-arm commented 2 months ago

Remove configurable endianness from target attributes for RteModel conditions compliance. The attribute was being removed before CPRJ generation but not before the model processing leading to a misaligned cbuild.yml.

codecov[bot] commented 2 months ago

Codecov Report

Attention: Patch coverage is 50.00000% with 3 lines in your changes are missing coverage. Please review.

Project coverage is 63.05%. Comparing base (b3cce86) to head (45dee40).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #1485 +/- ## ========================================== - Coverage 63.06% 63.05% -0.01% ========================================== Files 161 161 Lines 30569 30573 +4 Branches 18706 18710 +4 ========================================== Hits 19277 19277 + Misses 7533 7529 -4 - Partials 3759 3767 +8 ``` | [Flag](https://app.codecov.io/gh/Open-CMSIS-Pack/devtools/pull/1485/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Open-CMSIS-Pack) | Coverage Δ | | |---|---|---| | [projmgr-cov](https://app.codecov.io/gh/Open-CMSIS-Pack/devtools/pull/1485/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Open-CMSIS-Pack) | `80.09% <50.00%> (-0.04%)` | :arrow_down: | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Open-CMSIS-Pack#carryforward-flags-in-the-pull-request-comment) to find out more. | [Files](https://app.codecov.io/gh/Open-CMSIS-Pack/devtools/pull/1485?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Open-CMSIS-Pack) | Coverage Δ | | |---|---|---| | [tools/projmgr/src/ProjMgrGenerator.cpp](https://app.codecov.io/gh/Open-CMSIS-Pack/devtools/pull/1485?src=pr&el=tree&filepath=tools%2Fprojmgr%2Fsrc%2FProjMgrGenerator.cpp&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Open-CMSIS-Pack#diff-dG9vbHMvcHJvam1nci9zcmMvUHJvak1nckdlbmVyYXRvci5jcHA=) | `88.88% <ø> (-0.09%)` | :arrow_down: | | [tools/projmgr/src/ProjMgrWorker.cpp](https://app.codecov.io/gh/Open-CMSIS-Pack/devtools/pull/1485?src=pr&el=tree&filepath=tools%2Fprojmgr%2Fsrc%2FProjMgrWorker.cpp&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Open-CMSIS-Pack#diff-dG9vbHMvcHJvam1nci9zcmMvUHJvak1ncldvcmtlci5jcHA=) | `86.46% <66.66%> (-0.03%)` | :arrow_down: | | [tools/projmgr/test/src/ProjMgrWorkerUnitTests.cpp](https://app.codecov.io/gh/Open-CMSIS-Pack/devtools/pull/1485?src=pr&el=tree&filepath=tools%2Fprojmgr%2Ftest%2Fsrc%2FProjMgrWorkerUnitTests.cpp&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Open-CMSIS-Pack#diff-dG9vbHMvcHJvam1nci90ZXN0L3NyYy9Qcm9qTWdyV29ya2VyVW5pdFRlc3RzLmNwcA==) | `65.29% <33.33%> (-0.09%)` | :arrow_down: | ... and [1 file with indirect coverage changes](https://app.codecov.io/gh/Open-CMSIS-Pack/devtools/pull/1485/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Open-CMSIS-Pack)