Skip to content

Service layer abstraction#95

Merged
semohr merged 4 commits into
mainfrom
service_layer
May 11, 2026
Merged

Service layer abstraction#95
semohr merged 4 commits into
mainfrom
service_layer

Conversation

@semohr
Copy link
Copy Markdown
Contributor

@semohr semohr commented May 5, 2026

This pull request introduces a new plugin architecture for service discovery and registration. It replaces the previous hardcoded service mappings with a dynamic, extensible Service/ServiceRegistry system. Each service now registers itself as a subclass of Service, enabling automatic discovery, improved validation, and easier extensibility.

@semohr semohr temporarily deployed to integration_tests May 5, 2026 16:10 — with GitHub Actions Inactive
@semohr semohr had a problem deploying to integration_tests May 5, 2026 16:10 — with GitHub Actions Failure
@semohr semohr temporarily deployed to integration_tests May 5, 2026 16:11 — with GitHub Actions Inactive
@semohr semohr had a problem deploying to integration_tests May 5, 2026 16:11 — with GitHub Actions Failure
@metasauce metasauce deleted a comment from github-actions Bot May 5, 2026
@codecov
Copy link
Copy Markdown

codecov Bot commented May 5, 2026

Codecov Report

❌ Patch coverage is 98.43750% with 1 line in your changes missing coverage. Please review.
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
plistsync/services/__init__.py 97.05% 0 Missing and 1 partial ⚠️

📢 Thoughts on this report? Let us know!

@semohr semohr temporarily deployed to integration_tests May 6, 2026 14:50 — with GitHub Actions Inactive
@semohr semohr had a problem deploying to integration_tests May 6, 2026 14:50 — with GitHub Actions Failure
@semohr semohr temporarily deployed to integration_tests May 6, 2026 14:55 — with GitHub Actions Inactive
@semohr semohr requested a review from pSpitzner May 6, 2026 14:55
@semohr semohr temporarily deployed to integration_tests May 7, 2026 15:01 — with GitHub Actions Inactive
@semohr semohr temporarily deployed to integration_tests May 7, 2026 15:01 — with GitHub Actions Inactive
@semohr semohr merged commit 6734d15 into main May 11, 2026
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant