-
Notifications
You must be signed in to change notification settings - Fork 32
WIP:♻️ [Maintenance] Mai/todo or not todo #8465
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
base: master
Are you sure you want to change the base?
Conversation
…chemy.ext.asyncio 🗑️ Remove unused files.py dependency 📝 Clean up JobInputs model by removing outdated TODO comment
…comments in services_consume_filetypes.py and httpx_calls_capture.py; adjust webserver_service fixture scope in simcore_webserver_service.py
…ve outdated TODO comments in traefik_service.py and osparc_config.py; clean up settings.py by removing unnecessary TODOs
…inputs function; enhance type conversion using Pydantic smart unions and improve logging for unresolved parameters
…tency; enhance validation logic in validate_callable_signature function and add comprehensive tests for various scenarios
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #8465 +/- ##
==========================================
- Coverage 87.69% 85.13% -2.57%
==========================================
Files 1983 860 -1123
Lines 77277 32442 -44835
Branches 1333 966 -367
==========================================
- Hits 67771 27618 -40153
+ Misses 9107 4504 -4603
+ Partials 399 320 -79
Continue to review full report in Codecov by Sentry.
🚀 New features to boost your workflow:
|
|


What do these changes do?
Related issue/s
How to test
Dev-ops