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
158 stars 85 forks source link

Feature/509 remove osgi defines #679

Closed pnoltes closed 8 months ago

pnoltes commented 8 months ago

This PR removes the OSGI_ define constants and the usage of these constants.

Note that all OSGI_ define constants were already deprecated.

This PR also:

codecov-commenter commented 8 months ago

Codecov Report

Merging #679 (0a3b720) into master (5cdd18d) will increase coverage by 0.00%. The diff coverage is 88.88%.

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

@@           Coverage Diff           @@
##           master     #679   +/-   ##
=======================================
  Coverage   81.68%   81.69%           
=======================================
  Files         252      252           
  Lines       32633    32589   -44     
=======================================
- Hits        26657    26624   -33     
+ Misses       5976     5965   -11     
Files Coverage Δ
bundles/logging/log_admin/src/celix_log_admin.c 99.76% <100.00%> (ø)
...les/pubsub/pubsub_admin_tcp/src/pubsub_tcp_admin.c 50.50% <100.00%> (ø)
...b/pubsub_admin_tcp/src/pubsub_tcp_topic_receiver.c 68.87% <100.00%> (ø)
...sub/pubsub_admin_tcp/src/pubsub_tcp_topic_sender.c 84.01% <100.00%> (ø)
...ubsub_admin_websocket/src/pubsub_websocket_admin.c 42.47% <100.00%> (ø)
...in_websocket/src/pubsub_websocket_topic_receiver.c 67.47% <100.00%> (ø)
...les/pubsub/pubsub_admin_zmq/src/pubsub_zmq_admin.c 52.96% <100.00%> (ø)
...b/pubsub_admin_zmq/src/pubsub_zmq_topic_receiver.c 76.64% <100.00%> (ø)
...sub/pubsub_admin_zmq/src/pubsub_zmq_topic_sender.c 84.05% <100.00%> (ø)
..._protocol_wire_v1/src/ps_wire_protocol_activator.c 100.00% <100.00%> (ø)
... and 36 more

... and 1 file with indirect coverage changes

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more