-
Notifications
You must be signed in to change notification settings - Fork 32
♻️Maintenance: remove circular dependency #8245
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
♻️Maintenance: remove circular dependency #8245
Conversation
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## master #8245 +/- ##
==========================================
+ Coverage 88.01% 89.82% +1.81%
==========================================
Files 1916 1508 -408
Lines 74117 62257 -11860
Branches 1301 498 -803
==========================================
- Hits 65232 55925 -9307
+ Misses 8494 6203 -2291
+ Partials 391 129 -262
Continue to review full report in Codecov by Sentry.
🚀 New features to boost your workflow:
|
matusdrobuliak66
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks.
|



What do these changes do?
In order to make the simultaneous access using the new
is-collaborativelabel work, this pre-requisiste is needed.This should have no effect on the functionality.
This basically moves models from models-library/docker.py to services_metadata_runtime.py
Related issue/s
How to test
Dev-ops