radical-cybertools / radical.utils

Utility classes and tools for various radical projects
Other
8 stars 6 forks source link

Feature/use registry #385

Closed andre-merzky closed 1 year ago

andre-merzky commented 1 year ago

This PR supports https://github.com/radical-cybertools/radical.pilot/pull/2926 by removing the dependency from config files in the ZMQ bridge classes.

codecov[bot] commented 1 year ago

Codecov Report

:exclamation: No coverage uploaded for pull request base (devel_nodb_2@402b4f8). Click here to learn what that means. The diff coverage is n/a.

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

@@               Coverage Diff               @@
##             devel_nodb_2     #385   +/-   ##
===============================================
  Coverage                ?   61.72%           
===============================================
  Files                   ?       60           
  Lines                   ?     6777           
  Branches                ?        0           
===============================================
  Hits                    ?     4183           
  Misses                  ?     2594           
  Partials                ?        0           

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

mturilli commented 1 year ago

Apparently, we have new failures in the required tests. Do we need to fix them before merging?

mturilli commented 1 year ago

Two tests failing. Need fixing before merging into nodb_2