You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@celix.apache.org by GitBox <gi...@apache.org> on 2022/12/13 02:57:07 UTC

[GitHub] [celix] PengZheng opened a new pull request, #458: Fix bundle dependency management in `celix_target_bundle_set_definition`

PengZheng opened a new pull request, #458:
URL: https://github.com/apache/celix/pull/458

   As the title suggests, dependency management in `celix_target_bundle_set_definition` is broken, which could lead to weird testing failures when doing TDD.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscribe@celix.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [celix] codecov-commenter commented on pull request #458: Fix bundle dependency management in `celix_target_bundle_set_definition`

Posted by GitBox <gi...@apache.org>.
codecov-commenter commented on PR #458:
URL: https://github.com/apache/celix/pull/458#issuecomment-1347680487

   # [Codecov](https://codecov.io/gh/apache/celix/pull/458?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) Report
   > Merging [#458](https://codecov.io/gh/apache/celix/pull/458?src=pr&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) (a622ed4) into [master](https://codecov.io/gh/apache/celix/commit/363c1a9225fd21a8abac36dbf338726dabf1c292?el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) (363c1a9) will **increase** coverage by `0.01%`.
   > The diff coverage is `n/a`.
   
   ```diff
   @@            Coverage Diff             @@
   ##           master     #458      +/-   ##
   ==========================================
   + Coverage   74.48%   74.49%   +0.01%     
   ==========================================
     Files         199      199              
     Lines       29615    29615              
   ==========================================
   + Hits        22060    22063       +3     
   + Misses       7555     7552       -3     
   ```
   
   
   | [Impacted Files](https://codecov.io/gh/apache/celix/pull/458?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) | Coverage Δ | |
   |---|---|---|
   | [libs/utils/src/hash\_map.c](https://codecov.io/gh/apache/celix/pull/458/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-bGlicy91dGlscy9zcmMvaGFzaF9tYXAuYw==) | `94.10% <0.00%> (+0.84%)` | :arrow_up: |
   
   :mega: We’re building smart automated test selection to slash your CI/CD build times. [Learn more](https://about.codecov.io/iterative-testing/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscribe@celix.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [celix] PengZheng merged pull request #458: Fix bundle dependency management in `celix_target_bundle_set_definition`

Posted by GitBox <gi...@apache.org>.
PengZheng merged PR #458:
URL: https://github.com/apache/celix/pull/458


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscribe@celix.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org