codecov-commenter
commented
1 year ago Codecov Report
Merging #601 (df6efe0) into master (2683e8b) will increase coverage by
1.14%. The diff coverage is36.36%.:exclamation: Current head df6efe0 differs from pull request most recent head 4ce9a10. Consider uploading reports for the commit 4ce9a10 to get more accurate results
@@ Coverage Diff @@
## master #601 +/- ##
==========================================
+ Coverage 78.56% 79.71% +1.14%
==========================================
Files 260 256 -4
Lines 35415 34686 -729
==========================================
- Hits 27824 27649 -175
+ Misses 7591 7037 -554 | Files Changed | Coverage Δ | |
|---|---|---|
| libs/framework/src/bundle.c | 70.76% <ø> (-0.16%) |
:arrow_down: |
| libs/framework/src/manifest_parser.c | 92.85% <ø> (+60.79%) |
:arrow_up: |
| libs/framework/src/module.c | 83.00% <0.00%> (+8.63%) |
:arrow_up: |
| libs/framework/src/celix_launcher.c | 92.35% <100.00%> (ø) |
|
| libs/framework/src/framework.c | 84.86% <100.00%> (-0.11%) |
:arrow_down: |
... and 8 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more
This PR removes resolver and related codes from the framework, including:
Export-LibraryandImport-Librarymanifest parsing code (thus it fixes #531)export_importexampleThe removal should be safe, considering "export-library/import-library" has never been production ready. It can be viewed as a preparatory step towards reintroducing Requirement-Capability Model (#511).