search

apache / celix

Apache Celix is a framework for C and C++14 to develop dynamic modular software applications using component and in-process service-oriented programming.
https://celix.apache.org/
Apache License 2.0
165 stars 86 forks source link

Feature/509 remove pubsub #703

Closed pnoltes closed 9 months ago

pnoltes commented 9 months ago

This PR removes the pubsub feature from Apache Celix.

As stated in #509, pubsub is not part of the OSGi specification, and I believe it would be better for Apache Celix to focus on features from the OSGi spec, perhaps with a few additional small features.

This change should simplify the maintenance of Apache Celix and contribute to its growth as an even more stable framework.

Note that the pubsub feature is still available in the Apache Celix 2.4.x release. Should there be renewed interest in the future, we can consider reintroducing it based on community needs and contributions.

It fixes #466 It fixes #520 It fixes #295 It fixes #222 It fixes #221 It fixes #205 It fixes #123

codecov-commenter commented 9 months ago

Codecov Report

Attention: 27 lines in your changes are missing coverage. Please review.

Comparison is base (2a5058f) 84.25% compared to head (6856982) 88.26%.

:exclamation: Current head 6856982 differs from pull request most recent head efad168. Consider uploading reports for the commit efad168 to get more accurate results

Files Patch % Lines
...ration/src/TestExportImportRemoteServiceFactory.cc 83.83% 27 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #703 +/- ## ========================================== + Coverage 84.25% 88.26% +4.01% ========================================== Files 248 215 -33 Lines 31124 24562 -6562 ========================================== - Hits 26222 21679 -4543 + Misses 4902 2883 -2019 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.